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

Contributor docs: loosen restrictions on caniuse correspondence #1915

Merged
merged 3 commits into from
Oct 14, 2024

Conversation

ddbeck
Copy link
Collaborator

@ddbeck ddbeck commented Oct 7, 2024

Right now, we spend quite a lot of time reviewing and correcting discrepancies with caniuse. Only some of those discrepancies are visible to developers (right now), so I'm proposing relaxing the rules such that we care only about:

  • Matching caniuse on supported/unsupported across the core browser set
  • Matching caniuse on the keystone version

This change means we will (temporarily) overlook discrepancies for versions that do not affect the overall status or Baseline low date.

In the medium term, we ought to do #1880 and burn down its errors and warnings.

I spoke with @Fyrd today about this idea and got his nod of approval for this approach.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Oct 7, 2024
@ddbeck ddbeck mentioned this pull request Oct 10, 2024
@jamesnw jamesnw mentioned this pull request Oct 10, 2024
docs/guidelines.md Outdated Show resolved Hide resolved
docs/guidelines.md Outdated Show resolved Hide resolved
Co-authored-by: Patrick Brosset <patrick.brosset@microsoft.com>
docs/guidelines.md Outdated Show resolved Hide resolved
@ddbeck ddbeck merged commit 7d3bca1 into web-platform-dx:main Oct 14, 2024
3 checks passed
@ddbeck ddbeck deleted the caniuse-guideline-looser branch October 14, 2024 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants