Skip to content

arrange: confirm before discarding changes on quit - #10185

Open
pederbe wants to merge 1 commit into
jj-vcs:mainfrom
pederbe:learn/jj-9243-arrange-quit
Open

pederbe wants to merge 1 commit into
jj-vcs:mainfrom
pederbe:learn/jj-9243-arrange-quit

Conversation

@pederbe

@pederbe pederbe commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Fixes #9243.

Checklist

If applicable:

  • I have updated CHANGELOG.md
  • I have updated the documentation (README.md, docs/, demos/)
  • I have updated the config schema (cli/src/config-schema.json)
  • I have added/updated tests to cover my changes
  • I fully understand the code that I am submitting (what it does,
    how it works, how it's organized), including any code drafted by an LLM.
  • For any prose generated by an LLM, I have proof-read and copy-edited with
    an eye towards deleting anything that is irrelevant, clarifying anything
    that is confusing, and adding details that are relevant. This includes,
    for example, commit descriptions, PR descriptions, and code comments.

@pederbe
pederbe marked this pull request as ready for review September 13, 2026 08:40
@pederbe
pederbe requested a review from a team as a code owner September 13, 2026 08:40
Quitting arrange currently reports an error and discards pending edits without asking. Exit successfully on quit, and ask before discarding changes to commit actions or parents. Ignore navigation and reverted edits when deciding whether to ask.

Fixes jj-vcs#9243
@pederbe
pederbe force-pushed the learn/jj-9243-arrange-quit branch from c818c23 to 1d07566 Compare September 14, 2026 12: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.

Quit from arrange should not be an error but ask if something is discarded

1 participant