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

feat(schemas/browsers)!: merge accepts_{flags,webextension} fields into accepts array #25825

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Jan 31, 2025

Summary

Merges the accepts_flags and accepts_webextensions fields in the browser schema into a single accepts array that accepts the enum values flags and webextensions.

Important

This is a breaking change, as we expose these fields in the built data.json.

Test results and supporting details

Prepares for #25804, which adds webdriver-bidi.

Related issues

@github-actions github-actions bot added schema Isses or pull requests regarding the JSON schema files used in this project. infra Infrastructure issues (npm, GitHub Actions, releases) of this project docs Issues or pull requests regarding the documentation of this project. linter Issues or pull requests regarding the tests / linter of the JSON files. data:browsers Data about browsers (versions, release dates, etc). This data is used for validation. scripts Issues or pull requests regarding the scripts in scripts/. labels Jan 31, 2025
@caugner caugner changed the title [Schema] Merge browser accepts_* fields into accepts array [Schema] Merge browser accepts_{flags,webextension} fields into accepts array Jan 31, 2025
@github-actions github-actions bot added the size:l [PR only] 101-1000 LoC changed label Jan 31, 2025
@caugner caugner added the semver-major-bump A change that is potentially breaking for consumers label Jan 31, 2025
@caugner caugner changed the title [Schema] Merge browser accepts_{flags,webextension} fields into accepts array feat(schemas/browsers)!: merge accepts_{flags,webextension} fields into accepts array Jan 31, 2025
Copy link
Contributor

@queengooborg queengooborg left a comment

Choose a reason for hiding this comment

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

Considering that we're adding more flags for featuresets browsers allow, this makes sense.

Small nit: perhaps it would make more sense to make this a dictionary instead, so that we can add version numbers for when a featureset was introduced?

@caugner caugner added this to the BCD 6.0 milestone Feb 18, 2025
@caugner
Copy link
Contributor Author

caugner commented Feb 24, 2025

Small nit: perhaps it would make more sense to make this a dictionary instead, so that we can add version numbers for when a featureset was introduced?

@queengooborg I'm open to discussing the use case behind making this a dictionary, but I would prefer to start with this basic approach, to avoid scope creep.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:browsers Data about browsers (versions, release dates, etc). This data is used for validation. docs Issues or pull requests regarding the documentation of this project. infra Infrastructure issues (npm, GitHub Actions, releases) of this project linter Issues or pull requests regarding the tests / linter of the JSON files. schema Isses or pull requests regarding the JSON schema files used in this project. scripts Issues or pull requests regarding the scripts in scripts/. semver-major-bump A change that is potentially breaking for consumers size:l [PR only] 101-1000 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants