-
Notifications
You must be signed in to change notification settings - Fork 698
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
Update & consolidate some repo dependencies #7090
Conversation
penalosa
commented
Oct 24, 2024
•
edited
Loading
edited
- Moves more deps to the catalog, including undici & typescript
- Updating Typescript + @types/node@18 to the latest
- Updated ws@7.x to the latest patch version in Wrangler's bundle, resolving a semgrep issue (included via ink)
- Tests
- TODO (before merge)
- Tests included
- Tests not necessary because: covered by existing tests
- E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)
- I don't know
- Required
- Not required because: covered by existing tests
- Public documentation
- TODO (before merge)
- Cloudflare docs PR(s):
- Documentation not necessary because: dep upgrades
|
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/11522038735/npm-package-wrangler-7090 You can reference the automatically updated head of this PR with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/7090/npm-package-wrangler-7090 Or you can use npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11522038735/npm-package-wrangler-7090 dev path/to/script.js Additional artifacts:npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11522038735/npm-package-create-cloudflare-7090 --no-auto-update npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11522038735/npm-package-cloudflare-kv-asset-handler-7090 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11522038735/npm-package-miniflare-7090 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11522038735/npm-package-cloudflare-pages-shared-7090 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11522038735/npm-package-cloudflare-vitest-pool-workers-7090 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11522038735/npm-package-cloudflare-workers-editor-shared-7090 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11522038735/npm-package-cloudflare-workers-shared-7090 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11522038735/npm-package-cloudflare-workflows-shared-7090 Note that these links will no longer work once the GitHub Actions artifact expires.
Please ensure constraints are pinned, and |
d0a6035
to
768d1b9
Compare
Fixes #7094 |
00d6259
to
cb24802
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a great improvement! thank you @penalosa