Skip to content

WIP: Add a persisted telemetry setting to mapbox config - #47

Draft
zmofei wants to merge 1 commit into
telemetry-eventsfrom
telemetry-config
Draft

zmofei wants to merge 1 commit into
telemetry-eventsfrom
telemetry-config

Conversation

@zmofei

@zmofei zmofei commented Sep 24, 2026

Copy link
Copy Markdown
Member

WIP — stacked on #46. Review that first; this PR's base is telemetry-events.

Adds a persisted telemetry setting to mapbox config, so mapbox config set telemetry off turns the telemetry event off in every shell, the way MAPBOX_CLI_NO_TELEMETRY=1 does for one.

  • config get/set/unset/list accept telemetry, and config list now reports it alongside update-check.
  • The setting is read again at exit, so the run that turns it off doesn't report itself.
  • It only covers the event. User-Agent markers still follow MAPBOX_CLI_NO_TELEMETRY alone. Should the config key cover them too?

@zmofei
zmofei force-pushed the telemetry-config branch 2 times, most recently from 34494a5 to 4bd4118 Compare September 24, 2026 14:09
mapbox config set telemetry off stops the run's telemetry event in every
shell, beside MAPBOX_CLI_NO_TELEMETRY for one. The setting is read again
at exit, so the run that turns it off does not report itself.
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.

1 participant