-
Notifications
You must be signed in to change notification settings - Fork 719
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
Remove rocket column #174
Remove rocket column #174
Conversation
I think either we keep the columns everywhere, or remove them everywhere. The columns being temporarily unused doesn’t mean they won’t be used the next time something is ready for advancement. |
I'll keep this PR open and update it as soon as we'll have a decision on #173 |
d6ac956
to
9a027a2
Compare
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.
I appreciate the rebase and updates; but let's hold off on merging this until it's clearer that there's consensus on it.
9a027a2
to
58fe3d1
Compare
58fe3d1
to
16d8a0d
Compare
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 has been open awhile with no objections from anyone; time to merge it.
If a delegate objects in the future, we can revert.
This patch removes columns with rocket only from those pages where they aren't used (see #161) and keeps that column for active stage 1+ proposals page till rockets there will be replaced by #173 (WIP)
stage-0-proposals.md rendered version
ecma402/README.md rendered version
ecma402/stage-0-proposals.md rendered version
closes #161