Skip to content

docs: drop the Claude-config wording from uninstall#126

Merged
GordonBeeming merged 1 commit into
mainfrom
gb/uninstall-drop-claude-wording
Jun 29, 2026
Merged

docs: drop the Claude-config wording from uninstall#126
GordonBeeming merged 1 commit into
mainfrom
gb/uninstall-drop-claude-wording

Conversation

@GordonBeeming

Copy link
Copy Markdown
Owner

Summary

  • Removes the "Your Claude config (~/.claude) is left untouched" line from the uninstall output (CLI + remote uninstall.sh / uninstall.ps1), and the matching notes in the README "Uninstalling" section, docs/uninstall.md, and the --purge help text.
  • It read as odd, mid-uninstall noise. Behaviour is unchanged — uninstall never removed ~/.claude (it's simply not in the removal list); only the wording is gone.

Test plan

  • dotnet build green; uninstall surfaces have no remaining Claude-config mentions.
  • bash -n uninstall.sh + PowerShell parse pass.
  • Ran uninstall.sh — intro output no longer shows the Claude line.

The "Your Claude config (~/.claude) is left untouched" line in the uninstall
output (and the matching notes in the README, docs, and --purge help) read as
odd, mid-uninstall noise. Behaviour is unchanged — uninstall never removed
~/.claude (it's simply not in the removal list); only the wording is gone.

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: GitButler <gitbutler@gitbutler.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes all references, documentation notes, and console outputs indicating that the uninstall process does not touch the user's real Claude Code configuration directory (~/.claude) across the README, command-line options, shell integration code, and uninstall scripts. I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@GordonBeeming GordonBeeming marked this pull request as ready for review June 29, 2026 08:16
Copilot AI review requested due to automatic review settings June 29, 2026 08:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes references to “Claude config” (specifically ~/.claude) from uninstall messaging across the CLI, remote uninstall scripts, and documentation, without changing uninstall behavior.

Changes:

  • Removed ~/.claude/“Claude config” wording from uninstall output in uninstall.sh, uninstall.ps1, and the CLI uninstall flow.
  • Updated README and docs/uninstall.md to drop the corresponding uninstall note.
  • Simplified --purge help text to remove the parenthetical about ~/.claude.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
uninstall.sh Removes the ~/.claude “left untouched” line from uninstall output and header comment.
uninstall.ps1 Removes the ~/.claude “left untouched” line from uninstall output and header comment.
README.md Drops the ~/.claude mention from the Uninstalling section.
docs/uninstall.md Removes the ~/.claude note from the uninstall reference doc.
app/Infrastructure/ShellIntegration.cs Removes ~/.claude mention from uninstall XML doc + uninstall completion message.
app/Commands/Run/RunCommand.cs Updates --purge option description to remove the ~/.claude parenthetical.

@GordonBeeming GordonBeeming merged commit b928d8c into main Jun 29, 2026
32 checks passed
@GordonBeeming GordonBeeming deleted the gb/uninstall-drop-claude-wording branch June 29, 2026 08:27
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.

2 participants