-
Notifications
You must be signed in to change notification settings - Fork 2k
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 never-supported JavaScript features #6957
Conversation
"pipeline": { | ||
"__compat": { | ||
"description": "Pipeline operator (<code>|></code>)", | ||
"mdn_url": "https://developer.mozilla.org/docs/Web/JavaScript/Reference/Operators/Pipeline_operator", |
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 one is actually implemented https://bugzilla.mozilla.org/show_bug.cgi?id=1405943 but it's behind build flag and can't be enabled in release builds. I think it's safe to remove this.
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.
Since this PR was opened, the pipeline operator implementation has been removed from Firefox: https://bugzilla.mozilla.org/show_bug.cgi?id=1708546.
@Elchi3 Any sense of what should happen to the content for this one? I know there was a lot of buzz around the pipeline operator at one point, but I don't know if that's enough to keep the content around, if implementation is going backwards.
(The other stuff in this PR doesn't seem to have content, so it should be safe to remove.)
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.
hm, I'd say let's remove this for now then.
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.
Opened mdn/content#5394
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.
mdn/content#5394 has been merged, so it seems like it’s time to go ahead and merge this BCD PR too?
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.
Yeah, let's merge this 👍
This appears to be going backwards in terms of acceptance. For example: https://bugzilla.mozilla.org/show_bug.cgi?id=1708546 We're headed toward dropping it from BCD. See: mdn/browser-compat-data#6957 (comment)
This appears to be going backwards in terms of acceptance. For example: https://bugzilla.mozilla.org/show_bug.cgi?id=1708546 We're headed toward dropping it from BCD. See: mdn/browser-compat-data#6957 (comment)
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.
Thanks all! 👍
* Bump version to v3.3.6 * Add release note for #10646 * Add release note for #10581 * Add release note for #10685 * Add release note for #10691 * Add release note for #6957 * Add release note for #10721 * Add release note for #10695 * Add release note for #9821 * Add release note for #10681 * Add release note for #10725 * Add stats * Add release date * Wordsmith
A checklist to help your pull request get merged faster: