Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Week of Feb 5th edits #71

Merged
merged 1 commit into from
Feb 8, 2024
Merged

Week of Feb 5th edits #71

merged 1 commit into from
Feb 8, 2024

Conversation

duglin
Copy link
Contributor

@duglin duglin commented Feb 6, 2024

  • Add a "readonly" attribute to Resources in the model
  • rename "strict" to "deployed" in Endpoint spec
  • removed all "-" from attribute names in Endpoint spec
  • soften Schema spec to say schema versions SHOULD be backwards compatible,
    not MUST be
  • s/required/clientrequired/ & add "serverrequired" for model attributes
  • maps must be specified in their entirety when appearing as HTTP headers
  • move "origin" to Core spec
  • move "format" from Core to Endpoints, Message and Schema specs
  • clarify when self/latestversionurl have ?meta (or not)
  • clean-up some of the spec's model.json files

@duglin duglin force-pushed the feb5 branch 7 times, most recently from 5bc3770 to 03fd362 Compare February 8, 2024 15:46
- Add a "readonly" attribute to Resources in the model
- rename "strict" to "deployed" in Endpoint spec
- removed all "-" from attribute names in Endpoint spec
- soften Schema spec to say schema versions SHOULD be backwards compatible,
  not MUST be
- s/required/clientrequired/ & add "serverrequired" for model attributes
- maps must be specified in their entirety when appearing as HTTP headers
- move "origin" to Core spec
- move "format" from Core to Endpoints, Message and Schema specs
- clarify when self/latestversionurl have ?meta (or not)

Signed-off-by: Doug Davis <[email protected]>
@duglin
Copy link
Contributor Author

duglin commented Feb 8, 2024

Jon asked if we should go the other direction and add a word separator. We agreed to revisit this on the next Tues call

@duglin duglin merged commit 190e511 into xregistry:main Feb 8, 2024
2 checks passed
@duglin duglin deleted the feb5 branch February 8, 2024 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant