-
Notifications
You must be signed in to change notification settings - Fork 673
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
Pin chromium/tools/depot_tools to a specific commit #3251
Conversation
|
Codecov Report
@@ Coverage Diff @@
## main #3251 +/- ##
==========================================
- Coverage 75.17% 73.86% -1.32%
==========================================
Files 183 549 +366
Lines 11055 31361 +20306
Branches 2904 7418 +4514
==========================================
+ Hits 8311 23165 +14854
- Misses 2744 7926 +5182
- Partials 0 270 +270 |
A wrangler prerelease is available for testing. You can install this latest build in your project with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/5280953048/npm-package-wrangler-3251 You can reference the automatically updated head of this PR with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/3251/npm-package-wrangler-3251 Or you can use npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/5280953048/npm-package-wrangler-3251 dev path/to/script.js Additional artifacts:npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/5280953048/npm-package-cloudflare-pages-shared-3251 Note that these links will no longer work once the GitHub Actions artifact expires. |
This still has CI failures... |
a0f82be
to
535791d
Compare
535791d
to
502489a
Compare
ac14f4b
to
29a6837
Compare
Co-authored-by: Samuel Macleod <smacleod@cloudflare.com>
Fixes # [insert GH or internal issue number(s)].
What this PR solves / how to test:
It looks like we currently fail on tests because a new set of commits in https://chromium.googlesource.com/chromium/tools/depot_tools.git has broken our workflow in the DevTools publish action. We don't need to be on the bleeding edge here so we are pinning it to a commit that works for now.
Associated docs issue(s)/PR(s):
Author has included the following, where applicable:
Reviewer is to perform the following, as applicable: