Skip to content

Conversation

@stefanhaller
Copy link
Collaborator

@stefanhaller stefanhaller commented Jan 22, 2023

Add a new config option gui.skipRewordInEditorWarning. Addresses #2033.

No behavior change, just a preparation for the next commit.
Copy link
Owner

@jesseduffield jesseduffield left a comment

Choose a reason for hiding this comment

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

Nice work, just one comment :)

docs/Config.md Outdated
showIcons: false
commandLogSize: 8
splitDiff: 'auto' # one of 'auto' | 'always'
skipRewordInEditorPrompt: false # for skipping the confirmation before launching the reword editor
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
skipRewordInEditorPrompt: false # for skipping the confirmation before launching the reword editor
skipRewordInEditorWarning: false # for skipping the confirmation before launching the reword editor

This is more consistent with existing config keys

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

OK, makes sense. Here's a fixup: 73316fa.

(A fixup doesn't makes a huge amount of sense for this little PR, also because all the formatting changes, so I'm going to squash it right away.)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Argh, and the fixup wasn't even complete because I forgot to save Config.md. Sorry for the mess. But the current state should be good now.

@stefanhaller stefanhaller force-pushed the disable-reword-in-editor-prompt branch from 73316fa to 67fb28e Compare January 26, 2023 08:01
@github-actions
Copy link
Contributor

github-actions bot commented Jan 26, 2023

Uffizzi Preview Environment deployment-12345

☁️ https://app.uffizzi.com/github.com/jesseduffield/lazygit/pull/2384

📄 View Application Logs etc.

What is Uffizzi? Learn more

@jesseduffield jesseduffield merged commit 18db5ea into jesseduffield:master Jan 29, 2023
@jesseduffield
Copy link
Owner

Nice work @stefanhaller !

@stefanhaller stefanhaller deleted the disable-reword-in-editor-prompt branch January 29, 2023 09:54
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