Skip to content

devops: remove unused driver publishing#41518

Merged
Skn0tt merged 2 commits into
mainfrom
copilot/remove-driver-publishing
Jun 30, 2026
Merged

devops: remove unused driver publishing#41518
Skn0tt merged 2 commits into
mainfrom
copilot/remove-driver-publishing

Conversation

Copilot AI commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

The driver distribution is no longer used, so the driver publishing pipeline and its supporting scripts can be removed.

Changes

  • publish_release.yml: dropped the Azure CDN login + build & publish driver steps; renamed job publish-npm-and-driverpublish-npm and removed the allow-publish-driver-to-cdn environment (only NPM/trace-viewer publishing remains).
  • Scripts: deleted utils/build/build-playwright-driver.sh and utils/build/upload-playwright-driver.sh.
  • tests_secondary.yml: removed the build-playwright-driver CI job.
  • update-playwright-node.mjs: dropped the now-dangling driver build-script version bump; Docker Node.js rolling is unchanged.
  • print-api-json CLI command: removed the hidden command and the printApiJson helper from driver.ts, along with the now-unused path/packageRoot imports.

Co-authored-by: Skn0tt <14912729+Skn0tt@users.noreply.github.com>

@dgozman dgozman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

  • Let's also remove print-api-json cli command.
  • Let's wait until we publish the java port.

@Skn0tt

Skn0tt commented Jun 29, 2026

Copy link
Copy Markdown
Member

@copilot can you drop the print-api-json command?

Co-authored-by: Skn0tt <14912729+Skn0tt@users.noreply.github.com>

Copilot AI commented Jun 29, 2026

Copy link
Copy Markdown
Contributor Author

@copilot can you drop the print-api-json command?

Dropped the print-api-json command and its printApiJson helper (b3c30e0).

Copilot AI requested a review from Skn0tt June 29, 2026 10:36
@Skn0tt Skn0tt marked this pull request as ready for review June 29, 2026 11:14
@Skn0tt Skn0tt merged commit 7d22275 into main Jun 30, 2026
41 of 46 checks passed
@Skn0tt Skn0tt deleted the copilot/remove-driver-publishing branch June 30, 2026 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants