-
Notifications
You must be signed in to change notification settings - Fork 779
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
Pbd/update contrib guide #8110
base: main
Are you sure you want to change the base?
Pbd/update contrib guide #8110
Conversation
|
@@ -30,18 +30,23 @@ Any contributions you make will be via [Pull Requests](https://docs.github.com/e | |||
- Ensure you have [created an account](https://docs.github.com/en/get-started/onboarding/getting-started-with-your-github-account) on GitHub. | |||
- [Create your own fork](https://docs.github.com/en/get-started/quickstart/fork-a-repo) of [this repository](https://github.com/cloudflare/workers-sdk). | |||
- Clone your fork to your local machine | |||
|
|||
```sh | |||
> git clone https://github.com/<your-github-username>/workers-sdk |
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.
While you are editing this, remove the ">" - they make copy/paste a pain
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/13283601153/npm-package-wrangler-8110 You can reference the automatically updated head of this PR with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/8110/npm-package-wrangler-8110 Or you can use npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-wrangler-8110 dev path/to/script.js Additional artifacts:cloudflare-workers-bindings-extension: wget https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-cloudflare-workers-bindings-extension-8110 -O ./cloudflare-workers-bindings-extension.0.0.0-vde4bdca9f.vsix && code --install-extension ./cloudflare-workers-bindings-extension.0.0.0-vde4bdca9f.vsix create-cloudflare: npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-create-cloudflare-8110 --no-auto-update @cloudflare/kv-asset-handler: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-cloudflare-kv-asset-handler-8110 miniflare: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-miniflare-8110 @cloudflare/pages-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-cloudflare-pages-shared-8110 @cloudflare/unenv-preset: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-cloudflare-unenv-preset-8110 @cloudflare/vite-plugin: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-cloudflare-vite-plugin-8110 @cloudflare/vitest-pool-workers: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-cloudflare-vitest-pool-workers-8110 @cloudflare/workers-editor-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-cloudflare-workers-editor-shared-8110 @cloudflare/workers-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-cloudflare-workers-shared-8110 @cloudflare/workflows-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13283601153/npm-package-cloudflare-workflows-shared-8110 Note that these links will no longer work once the GitHub Actions artifact expires.
Please ensure constraints are pinned, and |
Minor tweaks...
Given that the root
pnpm test
seems to fail right now, I assume no one is actually using it.