-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
Linkify stability policy #941
Conversation
Fixes #930 Also clarifies that [!IMPORANT] blocks are normative.
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.
Just one item
Updates to this specification will not remove any options or option values | ||
defined in the default registry. | ||
Updates to this specification will not remove any _options_ or _option_ values | ||
defined in the default function registry. | ||
|
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.
One additional item. We should add the following clause for clarity.
Updates to this specification may **deprecate** _functions_, _options_, and _operands_ defined in the default registry, as is standard practice in Unicode specifications. This is to indicate that the use of those _functions_, _options_, or _operands_ is discouraged. However, they are never removed, and implementations can continue to support them. |
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.
We already say this (line 146), except we don't mention operands in the stability policy and, of course, it's much less explanatory. Note that the stability policy is about what we won't do.
Making this change would be normative and require WG consensus, so I'll change the tags and remove fast-track until we discuss Monday.
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.
Sounds good
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.
Eh, this is a completely separate change from this PR. It really should be handled separately, but we're going to feature-creep it into this same change once again, yes?
Something like this was previously discussed here: #883 (comment), and the spec text already includes this, which is in an [!IMPORTANT]
block and therefore normative:
message-format-wg/spec/README.md
Line 142 in f3344b5
> - Future versions may deprecate (but not remove) keywords, functions, options, or option values. |
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.
Eh, this is a completely separate change from this PR. It really should be handled separately, but we're going to feature-creep it into this same change once again, yes?
Under normal circumstances, I would insist on it. But I'm hoping to be done tomorrow.
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.
The current text looks good. I don't think the added deprecation item is necessary, as we already mention it later in the text. If there's something missing from that (such as explicitly allowing for deprecating operands), it ought to be modified in place.
Fixes #930
Also clarifies that [!IMPORANT] blocks are normative.