-
Notifications
You must be signed in to change notification settings - Fork 723
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
feat(wrangler): enable telemetry by default #7291
base: main
Are you sure you want to change the base?
Conversation
|
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/11976967207/npm-package-wrangler-7291 You can reference the automatically updated head of this PR with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/7291/npm-package-wrangler-7291 Or you can use npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11976967207/npm-package-wrangler-7291 dev path/to/script.js Additional artifacts:npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11976967207/npm-package-create-cloudflare-7291 --no-auto-update npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11976967207/npm-package-cloudflare-kv-asset-handler-7291 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11976967207/npm-package-miniflare-7291 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11976967207/npm-package-cloudflare-pages-shared-7291 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11976967207/npm-package-cloudflare-vitest-pool-workers-7291 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11976967207/npm-package-cloudflare-workers-editor-shared-7291 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11976967207/npm-package-cloudflare-workers-shared-7291 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11976967207/npm-package-cloudflare-workflows-shared-7291 Note that these links will no longer work once the GitHub Actions artifact expires.
Please ensure constraints are pinned, and |
d6ca47a
to
4799bba
Compare
* add telemetry commands * changeset * fix and test dates * update changeset * add global/project status * default true * remove changeset * update wrangler telemetry status
* stop collecting userId in telemetry Co-authored-by: emily-shen <emily-shen@users.noreply.github.com> * implement telemetry collection * infer errorType based on the constructor name * implement common event properties * log common event properties Co-authored-by: Edmund Hung <me@edmund.dev> * respect metric enabled/disabled * remove dispatcher.identify * include SPARROW_SOURCE_KEY in PR pre-release build * fix tests * ensure debug log covers the request failed message * replace SPARROW_SOURCE_KEY regardless whethe env exists --------- Co-authored-by: Edmund Hung <edmund@cloudflare.com> Co-authored-by: emily-shen <emily-shen@users.noreply.github.com> Co-authored-by: Edmund Hung <me@edmund.dev>
4799bba
to
153fd60
Compare
* add banner * abort if telemetry disable * basic sendNewEvent tests * banner tests * settle promises before exiting * remove unnecessary banner logic * just check if version is different
* isCI and isNonInteractive * add argsUsed and argsCombination * don't include args if value is false * redact arg values if string * lint * isNonInteractive -> isInteractive
Fixes DEVX-1501.
Describe your change...
wrangler telemetry
commands and update existing info on telemetry collection cloudflare-docs#18350