Skip to content
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

Editorial: drop Changes section #161

Closed
wants to merge 1 commit into from
Closed

Editorial: drop Changes section #161

wants to merge 1 commit into from

Conversation

marcoscaceres
Copy link
Member

@marcoscaceres marcoscaceres commented May 2, 2024

This section adds limited value that one can't get from the actual commit history. The document already links to the commit history in the head. It's also hassle for editors to maintain.

We should as Bikeshed to generate this section automatically once it supports such a thing... same a ReSpec does, with link to the appropriate PRs etc.


Preview | Diff

@marcoscaceres marcoscaceres requested a review from reillyeon May 2, 2024 02:02
@anssiko
Copy link
Member

anssiko commented May 2, 2024

This section is in place to satisfy https://www.w3.org/2023/Process-20231103/#transition-reqs for our upcoming CR Snapshot publication as communicated in the CfC sent to both the groups: https://lists.w3.org/Archives/Public/public-device-apis/2024Mar/0000.html

In a chair capacity, I'll ask the editors to make sure this section is updated to reflect the latest changes, and not removed. Please open a new PR for that. Thank you for your contributions that help explain this specification's journey to its readers.

@anssiko anssiko closed this May 2, 2024
@anssiko anssiko deleted the changes branch May 2, 2024 10:15
@himorin
Copy link
Contributor

himorin commented May 2, 2024

@marcoscaceres I support an idea that bikeshed should have a feature to auto-generate changes or commit history section, with breaking into periods like during WD or between two CRS, but I cannot support this PR to remove existing list before bikeshed supports and configuration included in this PR.

@reillyeon
Copy link
Member

I filed a feature request for this in Bikeshed: speced/bikeshed#2847

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants