Skip to content

Conversation

oskarwires
Copy link
Contributor

Hi! I noticed that the message for the :wqa! command is contradictory. It states that we ignore unsaved changes, but the command actually forces everything to be saved. This is incorrect. This PR updates the message to make this clear.

Copy link
Contributor

@leo-unglaub leo-unglaub left a comment

Choose a reason for hiding this comment

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

Looks good to me

@oskarwires
Copy link
Contributor Author

Looks good to me

Thanks! Anyone else able to take a look so this can be landed please?

name: "write-quit-all!",
aliases: &["wqa!", "xa!"],
doc: "Write changes from all buffers to disk and close all views forcefully (ignoring unsaved changes).",
doc: "Forcefully write changes from all buffers to disk, creating necessary subdirectories, and close all views.",
Copy link
Member

Choose a reason for hiding this comment

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

force_write_all_quit also forcefully closes all views: both the write and the quit are forceful. For example if you open a scratch buffer and make changes without writing them to a file, :wqa will not quit while :wqa! will

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think the original message is still a bit unclear in regards to changes being made in saved buffers being written forcefully. When I first read the message, I understood it as implying that changes are not saved forcefully. Maybe a new version could be: Forcefully write changes from all buffers to disk, creating necessary subdirectories, and close all views (ignoring unsaved changes).?

Copy link
Member

Choose a reason for hiding this comment

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

That sounds good to me 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done 🙌 thanks for the feedback

@the-mikedavis the-mikedavis merged commit 282345c into helix-editor:master Oct 14, 2025
7 checks passed
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.

3 participants