Skip to content

ci(update-webdriver-bidi-data): split into modules#26887

Merged
caugner merged 3 commits intomainfrom
split-webdriver-bidi
May 23, 2025
Merged

ci(update-webdriver-bidi-data): split into modules#26887
caugner merged 3 commits intomainfrom
split-webdriver-bidi

Conversation

@caugner
Copy link
Copy Markdown
Contributor

@caugner caugner commented May 23, 2025

Summary

Updates the update-webdriver-bidi-data workflow to split up the webdriver/bidi.json file into multiple webdriver/bidi/<module>.json.

Test results and supporting details

This avoids a huge file, and makes it easier to review changes.

Test run succeeded:

Related issues

Instead of a single `webdriver/bidi.json` file, this will
create `webdriver/bidi/<module>.json` files, which makes
it easier to review changes.
@github-actions github-actions bot added infra Infrastructure issues (npm, GitHub Actions, releases) of this project size:xs [PR only] 0-6 LoC changed labels May 23, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 23, 2025

Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs).

@github-actions github-actions bot added scripts Issues or pull requests regarding the scripts in scripts/. size:s [PR only] 7-24 LoC changed and removed size:xs [PR only] 0-6 LoC changed labels May 23, 2025
Copy link
Copy Markdown
Contributor Author

@caugner caugner left a comment

Choose a reason for hiding this comment

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

Self-approval, as it 1.) fixes the split script, and 2.) just updates the WebDriver BiDi script to use that script.

@caugner caugner merged commit c3de40c into main May 23, 2025
13 checks passed
@caugner caugner deleted the split-webdriver-bidi branch May 23, 2025 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra Infrastructure issues (npm, GitHub Actions, releases) of this project scripts Issues or pull requests regarding the scripts in scripts/. size:s [PR only] 7-24 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant