Changes for week of Jan 20 #239
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
duglin
force-pushed
the
Jan20
branch
3 times, most recently
from
f6d2552 to
1fb31bf
Compare
duglin
force-pushed
the
Jan20
branch
6 times, most recently
from
53859a2 to
1ee1675
Compare
- Misc typos/clean-up - Require `target` to NOT point to a model entity that uses `ximport`. Meaning, `target` is not transitive - to keep things simple. - Tweak usage of `target`. Can be on any relative URL/URI, and if value starts with `/` then it must match the xid template. Similar for `relation` but `relation` must be an xid... but `target` is optional for all types. - Add "deprecated" to messages and schemas - define "xref" as a "relation" type with "target"="self-xid" - add "modelversion" as a model label - make ?compact convert self, COLLECTONSurl, defaultverionurl into relative URLs if they're part of the response - mandate that all relative URLs start with "/" and they're relative to the root path of the Registry - ban model changes that turn on hasDoc if Resources have RESOURCE* attribute extension names Fixes: xregistry#236 Fixes: xregistry#226 Fixes: xregistry#227 Fixes: xregistry#189 Signed-off-by: Doug Davis <[email protected]>
|
Approved on the 1/23 call |
duglin
added a commit
to duglin/xreg-spec
that referenced
this pull request
Feb 11, 2025
Changes for week of Jan 20 Signed-off-by: Doug Davis <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
targetto NOT point to a model entity that usesximport.Meaning,
targetis not transitive - to keep things simple.target. Can be on any relative URL/URI, and if value startswith
/then it must match the xid template. Similar forrelationbutrelationmust be an xid... buttargetis optional for all types.URLs if they're part of the response
root path of the Registry
extension names
Fixes: #236
Fixes: #226
Fixes: #227
Fixes: #189
Fixes: #238