Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update install.iss: rephrase menu item labels #507

Merged
merged 2 commits into from
Jul 7, 2023

Conversation

ivadham
Copy link
Contributor

@ivadham ivadham commented Jun 23, 2023

Windows context menu:

  • Git Bash Here -> Open Git Bash here
  • Git GUI Here -> Open Git GUI here

P.S. Today I learned what DCO sign-off is. Thank you!

• Git Bash Here -> Open Git Bash here
• Git GUI Here -> Open Git GUI here

Signed-off-by: Ali Mahdavi <ivadham@users.noreply.github.com>
Copy link
Member

@rimrul rimrul left a comment

Choose a reason for hiding this comment

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

First things first: ignore the missing dll CI job failures, those are unrelated.

The change does what was discussed in git-for-windows/git#4477 (comment), but that comment accidentally mislead you. These are the feature descriptions within the installer, not the text for the actual context menu entries. That text is defined here and here.
Could I ask you to modify these two lines as well and then ammend the commit and force push?

@ivadham
Copy link
Contributor Author

ivadham commented Jun 23, 2023

No worries. Thanks for the heads up!
Oh, shoot! I forgot to do DCO sign-off. I thought this was a one-time thing per PR.

@dscho
Copy link
Member

dscho commented Jun 23, 2023

that comment accidentally mislead you

Ooops. Sorry!

• Git Bash Here -> Open Git Bash here
• Git GUI Here -> Open Git GUI here

Signed-off-by: Ali Mahdavi <ivadham@users.noreply.github.com>
@ivadham
Copy link
Contributor Author

ivadham commented Jul 6, 2023

@rimrul, hey! Although I hate to disturb you by mentioning you, more than two weeks have passed since this "change request", and I have pushed the changes you asked for. It is done in two commits; maybe that's the reason why you have not been notified. Anyway, this PR requires a review. Thanks!

Copy link
Member

@dscho dscho 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, thank you for following through with it!

@dscho dscho merged commit 6f05fc9 into git-for-windows:main Jul 7, 2023
@dscho
Copy link
Member

dscho commented Jul 7, 2023

/add release note bug The labels of the File Explorer menu items installed by the Git for Windows installer have been aligned with what is customary ("Git Bash Here" instead of "Open Git Bash Here").

The workflow run was started

github-actions bot pushed a commit that referenced this pull request Jul 7, 2023
The labels of the File Explorer menu items installed by the Git for
Windows installer [have been
aligned](#507) with
what is customary ("Git Bash Here" instead of "Open Git Bash Here").

Signed-off-by: gitforwindowshelper[bot] <gitforwindowshelper-bot@users.noreply.github.com>
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