Skip to content

fix: telemetry disable command shows wrong success message#384

Merged
theoephraim merged 3 commits intomainfrom
copilot/bug-fix-telemetry-opt-out-message
Mar 12, 2026
Merged

fix: telemetry disable command shows wrong success message#384
theoephraim merged 3 commits intomainfrom
copilot/bug-fix-telemetry-opt-out-message

Conversation

Copy link
Contributor

Copilot AI commented Mar 12, 2026

  • Fix incorrect success message for varlock telemetry disable command
  • Add changeset for the patch fix
Original prompt

This section details on the original issue you should resolve

<issue_title>[BUG]: telemetry opt out displays the wrong output</issue_title>
<issue_description>### varlock version

0.4.0

Steps to reproduce

pnpm exec -- varlock telemetry disable
✅ Successfully enabled anonymous usage analytics

in the json it sets "telemetryDisabled": true

What is expected?

opt out message

What is actually happening?

value is written properly (functionality works) but message to user is incorrect

System Info

Any additional comments?

No response</issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

@changeset-bot
Copy link

changeset-bot bot commented Mar 12, 2026

🦋 Changeset detected

Latest commit: 6e9eeec

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 11 packages
Name Type
varlock Patch
@varlock/astro-integration Patch
@varlock/nextjs-integration Patch
@varlock/vite-integration Patch
@varlock/1password-plugin Patch
@varlock/aws-secrets-plugin Patch
@varlock/azure-key-vault-plugin Patch
@varlock/bitwarden-plugin Patch
@varlock/google-secret-manager-plugin Patch
@varlock/infisical-plugin Patch
@varlock/pass-plugin Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Co-authored-by: philmillman <3722211+philmillman@users.noreply.github.com>
Copilot AI changed the title [WIP] [BUG] Fix telemetry opt out output message fix: telemetry disable command shows wrong success message Mar 12, 2026
@philmillman philmillman marked this pull request as ready for review March 12, 2026 15:32
@philmillman philmillman requested a review from theoephraim March 12, 2026 15:32
@theoephraim
Copy link
Member

@copilot - add a changeset

Co-authored-by: theoephraim <1158956+theoephraim@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 12, 2026

@copilot - add a changeset

Added in 6e9eeec.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 12, 2026

Open in StackBlitz

npm i https://pkg.pr.new/varlock@384

commit: 6e9eeec

@theoephraim theoephraim merged commit 0642185 into main Mar 12, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: telemetry opt out displays the wrong output

3 participants