Skip to content

Conversation

@toddbaert
Copy link
Member

Adds make target that pulls the specified version of the flagd-json-schema and serves it at the appropriate location

@toddbaert toddbaert requested a review from a team May 27, 2024 20:32
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label May 27, 2024
@netlify
Copy link

netlify bot commented May 27, 2024

Deploy Preview for polite-licorice-3db33c canceled.

Name Link
🔨 Latest commit 8972336
🔍 Latest deploy log https://app.netlify.com/sites/polite-licorice-3db33c/deploys/6654ee01c1306100080cca25

Signed-off-by: Todd Baert <[email protected]>
toddbaert added a commit to open-feature/flagd-schemas that referenced this pull request May 27, 2024
This PR

- adds full versioning to the json schema (meaning the `$id` now has a
full version string in it, like:
`https://flagd.dev/schema/v0.2.5/flags.json`)
- adds automation to update this with new releases of the json module
- note there's no way to use the traditional release please versioning
with comment to update a substring of a json value, so I've had to hack
that in myself.

Next steps will be to:

- serve each version of the schema at flagd.dev:
open-feature/flagd#1321
- add support in providers for checking this

Signed-off-by: Todd Baert <[email protected]>
@toddbaert toddbaert requested a review from Kavindu-Dodan May 28, 2024 15:21
Copy link
Contributor

@Kavindu-Dodan Kavindu-Dodan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This conversation exist, but otherwise this looks good

@toddbaert
Copy link
Member Author

I'm going to close this. Reasoning here: open-feature/flagd-schemas#166

@toddbaert toddbaert closed this Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants