-
-
Notifications
You must be signed in to change notification settings - Fork 313
Add all changes since last release to Change Log #1360
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's impressively thorough! I don't think clarifications need a change log entry. I'd just include things that effect behavior.
<t>Consolidate and clarify the syntax and rationale for plain-name fragments</t> | ||
<t>"$id" MUST be an absolute-IRI, without any fragment, even an empty one</t> | ||
<t>Note that an empty string "$id" results in duplicate IRIs for different schemas</t> | ||
<t>Define empty schemas as empty (no longer allowing unrecognized keywords</t> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<t>Define empty schemas as empty (no longer allowing unrecognized keywords</t> | |
<t>Define empty schemas as empty (no longer allowing unrecognized keywords)</t> |
I agree though another riff on perhaps the same underlying suggestion is attempting to sort the list by some imprecise notion of "likelihood someone cares to read about it", which should be doable. E.g. |
Replaced by ☝️ |
As promised in Monday's call, here is the change log i had researched covering all changes to date (plus PR #1359 which was agreed to in principle on the call as well).