Skip to content

message "Would you like to stage all chages?"#109272

Merged
joaomoreno merged 1 commit intomicrosoft:masterfrom
fonsp:patch-1
Oct 28, 2020
Merged

message "Would you like to stage all chages?"#109272
joaomoreno merged 1 commit intomicrosoft:masterfrom
fonsp:patch-1

Conversation

@fonsp
Copy link
Contributor

@fonsp fonsp commented Oct 23, 2020

When clicking "commit" with no staged changes, the old message:

Would you like to automatically stage all your changes and commit them directly?

sounds like clicking Yes will do this automatically from now on, which is not the case. I removed automatically.

This PR fixes #

The old message,
```
Would you like to automatically stage all your changes and commit them directly?
```
sounds like clicking _Yes_ will do this _automatically_ from now on, which is not the case. I removed _automatically_.
@joaomoreno
Copy link
Member

While the current message might not be correct, the proposed change also fails to address the fact that a setting will be turned on to allow to commit unstaged changes directly, every time the user tries to commit and there are no staged changes. Any other proposals?

@fonsp
Copy link
Contributor Author

fonsp commented Oct 28, 2020

The current dialog is this:

image

The option Yes does not have a permanent effect, which is why I made this PR. It sounds like you are referring to the Always option, but in that case, the permanent effect is communicated by the button's label.

@joaomoreno
Copy link
Member

joaomoreno commented Oct 28, 2020

You're absolutely right! Thanks for the pointer and the PR!

@joaomoreno joaomoreno merged commit b6a02bb into microsoft:master Oct 28, 2020
@joaomoreno joaomoreno added this to the October 2020 milestone Oct 28, 2020
@joaomoreno joaomoreno added the git GIT issues label Oct 28, 2020
@fonsp fonsp deleted the patch-1 branch October 28, 2020 11:24
@github-actions github-actions bot locked and limited conversation to collaborators Dec 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

git GIT issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants