Skip to content

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 <[email protected]>
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 <[email protected]>
@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] <[email protected]>
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