Skip to content

Improve grammar of config setting description#28957

Merged
joaomoreno merged 1 commit intomicrosoft:masterfrom
gandreadis:patch-1
Jun 19, 2017
Merged

Improve grammar of config setting description#28957
joaomoreno merged 1 commit intomicrosoft:masterfrom
gandreadis:patch-1

Conversation

@gandreadis
Copy link
Contributor

This PR changes the config.enableSmartCommit setting description from

Commit all changes when there are not staged changes.

to

Commit all changes when there are no staged changes.

While the former version is technically correct, I consider the proposed version easier to read.

While the former version is technically correct, I consider the proposed version easier to read.
@mention-bot
Copy link

@gandreadis, thanks for your PR! By analyzing the history of the files in this pull request, we identified @joaomoreno and @bpasero to be potential reviewers.

@msftclas
Copy link

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Copy link
Member

@joaomoreno joaomoreno left a comment

Choose a reason for hiding this comment

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

Thanks!

@joaomoreno joaomoreno merged commit 6b9a527 into microsoft:master Jun 19, 2017
@gandreadis gandreadis deleted the patch-1 branch June 19, 2017 09:26
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants