-
Notifications
You must be signed in to change notification settings - Fork 120
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
Consolidate similar parts of REC revision #878
Merged
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
Revising a REC is a fairly complicated piece of the Process. The 4 subsections that deal with making revisions for class 1 through 4 are comparatively simple, but they had very strong similarities between class 1 and 2, and class 3 and 4. Consolidating the text not only makes the whole thing shorter, it also eliminates subtle differences of language that could leave people wondering about potential differences where none were intended or useful. This is a small part in addressing w3c#700
4bf5398
to
5d569c1
Compare
TallTed
reviewed
Jun 1, 2024
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.
Editorial, delayed by holiday...
Comment on lines
+4085
to
+4089
For [=Recommendations=] explicitly identified as [=allow new features|allowing new features=], | ||
tentative new features (see <a href="#correction-classes">class 4 changes</a>) | ||
<em class=rfc2119>may</em> be annotated in as [=candidate additions=], | ||
and <a href="#correction-classes">class 4 changes</a> may be normatively incorporated | ||
as for <a href="#correction-classes">class 3 changes</a> in [[#revised-rec-substantive]]. |
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.
Suggested change
For [=Recommendations=] explicitly identified as [=allow new features|allowing new features=], | |
tentative new features (see <a href="#correction-classes">class 4 changes</a>) | |
<em class=rfc2119>may</em> be annotated in as [=candidate additions=], | |
and <a href="#correction-classes">class 4 changes</a> may be normatively incorporated | |
as for <a href="#correction-classes">class 3 changes</a> in [[#revised-rec-substantive]]. | |
For [=Recommendations=] explicitly identified as [=allow new features|allowing new features=], | |
tentative new features (see <a href="#correction-classes">class 4 changes</a>) | |
<em class=rfc2119>may</em> be annotated as [=candidate additions=], | |
and <a href="#correction-classes">class 4 changes</a> may be normatively incorporated | |
in the same fashion as <a href="#correction-classes">class 3 changes</a> in [[#revised-rec-substantive]]. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Agenda+
Marks issues that are ready for discussion on the call
Closed: Accepted
The issue has been addressed, though not necessarily based on the initial suggestion
Topic: Simplifications
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.
Revising a REC is a fairly complicated piece of the Process. The 4 subsections that deal with making revisions for class 1 through 4 are comparatively simple, but they had very strong similarities between class 1 and 2, and class 3 and 4. Consolidating the text not only makes the whole thing shorter, it also eliminates subtle differences of language that could leave people wondering about potential differences where none were intended or useful.
This PR is an alternative to #865. It merges the 4 subsections into 3 rather than into 2, to keep "new features" distinct, so as not to make reading about class 3 changes more complicated for those who don't care about class 4.
This is a small part in addressing #700
Preview | Diff