Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Contributor docs: add caniuse usage guidance #1677

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

ddbeck
Copy link
Collaborator

@ddbeck ddbeck commented Aug 27, 2024

Fixes #1631

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Aug 27, 2024
Copy link
Collaborator

@jamesnw jamesnw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! This is helpful!

docs/guidelines.md Show resolved Hide resolved
- Do not set a `caniuse` value if the top-level headline status's first-supported release (for example, the value of `status.support.safari`) differs from Can I Use's first-supported release by:

- More than one release for releases since 2020
- More than one year for releases before 2020
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a way we could programmatically test this?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's likely, yes. I've filed #1680.

@ddbeck ddbeck merged commit c8086ac into web-platform-dx:main Sep 3, 2024
3 checks passed
@ddbeck ddbeck deleted the caniuse-usage-guidelines branch September 3, 2024 15:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clarifying the caniuse key
3 participants