Skip to content

Don't rewrite ~/.myclirc quotes on /fs or /fd - #2092

Merged
rolandwalker merged 1 commit into
mainfrom
RW/avoid-rewriting-myclirc-quoting-favorites
Aug 4, 2026
Merged

Don't rewrite ~/.myclirc quotes on /fs or /fd#2092
rolandwalker merged 1 commit into
mainfrom
RW/avoid-rewriting-myclirc-quoting-favorites

Conversation

@rolandwalker

Copy link
Copy Markdown
Contributor

Description

The user's own quoting, or lack of quoting, could be rewritten if saving or deleting a favorite query via the REPL. If the dotfile is kept under revision control, this can create needless churn.

We must manually triplequote multiline favorite queries on save, since quoting is completely disabled within the subclassed ConfigObj.

Incidentally strip semicolons from the end of favorite queries, along with whitespace.

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

The user's own quoting, or lack of quoting, could be rewritten if saving
or deleting a favorite query via the REPL.  If the dotfile is kept under
revision control, this can create needless churn.

We must manually triplequote multiline favorite queries on save, since
quoting is completely disabled within the subclassed ConfigObj.

Incidentally strip semicolons from the end of favorite queries, along
with whitespace.
@rolandwalker rolandwalker self-assigned this Aug 4, 2026
@rolandwalker
rolandwalker merged commit 9bbc5d5 into main Aug 4, 2026
11 checks passed
@rolandwalker
rolandwalker deleted the RW/avoid-rewriting-myclirc-quoting-favorites branch August 4, 2026 10:21
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