Skip to content

Suppress unsaved changes warning for qa!#9783

Merged
J-Fields merged 2 commits intomasterfrom
unknown repository
Oct 9, 2025
Merged

Suppress unsaved changes warning for qa!#9783
J-Fields merged 2 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link

@ghost ghost commented Oct 6, 2025

What this PR does / why we need it:

This fixes an issue where qa! could still show the "unsaved changes" warning.

We now iterate over all open tabs and call workbench.action.revertAndCloseActiveEditor for each one to ensure that changes are discarded without prompting, honoring the force-quit (!) option.

Which issue(s) this PR fixes:

Fixes #6742.

Hiroki Okada and others added 2 commits October 6, 2025 16:39
This fixes an issue where `qa!` could still show the "unsaved changes"
warning.

We now iterate over all open tabs and call
`workbench.action.revertAndCloseActiveEditor` for each one to ensure
that changes are discarded without prompting, honoring the force-quit
(`!`) option.

Fixes #6742.
Copy link
Member

@J-Fields J-Fields left a comment

Choose a reason for hiding this comment

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

Thanks!

@J-Fields J-Fields merged commit 7b8c4d9 into VSCodeVim:master Oct 9, 2025
1 check passed
@ghost ghost deleted the force-quit branch October 9, 2025 17: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.

:qa! does not suppress modified file warning and does not close window

1 participant