-
Notifications
You must be signed in to change notification settings - Fork 10
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
[CRS/202412] changes section from FPWD #306
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.
Thank you! I note the commit message titles are directly reused here without edits. An improvement might be to adjust wording in a few places, but I'm fine with this too.
@kenchris to check this looks good from an editor's perspective.
index.html
Outdated
<ul> | ||
<li><a href="https://github.com/w3c/compute-pressure/pull/302">Remove the limiting of rate as it is replace by obfuscation (PR #302)</a></li> | ||
<li><a href="https://github.com/w3c/compute-pressure/pull/300">Move at-risk to experimental on PressureSource/thermals (PR #300)</a></li> | ||
<li><a href="https://github.com/w3c/compute-pressure/pull/299">Make ownContributionEstimate experimental (PR #299)</a></li> |
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.
This entry and the next are not visible in the non-editors version, so can be removed
index.html
Outdated
<li><a href="https://github.com/w3c/compute-pressure/pull/302">Remove the limiting of rate as it is replace by obfuscation (PR #302)</a></li> | ||
<li><a href="https://github.com/w3c/compute-pressure/pull/300">Move at-risk to experimental on PressureSource/thermals (PR #300)</a></li> | ||
<li><a href="https://github.com/w3c/compute-pressure/pull/299">Make ownContributionEstimate experimental (PR #299)</a></li> | ||
<li><a href="https://github.com/w3c/compute-pressure/pull/298">Add support for ownContributionEstimate (PR #298)</a></li> |
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.
And this one
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.
Let's remove the two entries not visible in the non-editors version
Co-authored-by: Anssi Kostiainen <anssi.kostiainen@intel.com>
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.
LGTM
as title
not including consideration sections: 251, 247, 219, 216, 207
Preview | Diff