-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
JSON linter: don't allow version_added === version_removed #3531
Comments
Would you mind sharing a couple of these examples? |
I just submitted a PR that will fix them, but https://github.com/mdn/browser-compat-data/blob/master/api/RTCPeerConnection.json#L1999 |
+1 for this. Having a feature removed in the same version it was introduced in doesn’t make any sense! |
I thought this would be checked here https://github.com/mdn/browser-compat-data/blob/master/test/test-versions.js#L51 But maybe it is suitable to test this in the consistency tests as well. I haven't reviewed it yet. |
The test should be |
We have a couple of places here and there where version_added is invalidly the same as version_removed; we need to update the linter to catch this.
The text was updated successfully, but these errors were encountered: