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

Add SourceTree option in Windows Action Provider #128

Merged
merged 3 commits into from
Nov 6, 2020

Conversation

coenm
Copy link
Contributor

@coenm coenm commented Nov 5, 2020

Just found your project but I really like this tool. I also use SourceTree as a visual git client and added an extra action provider to start SourceTree with the current git repo selected.

I am aware of #114 to make this configurable but until then... you might want to add this.

@awaescher
Copy link
Owner

Hey @coenm, thank you very much, I'm happy to have you here.
Would you mind adding "Open in SourceTree" to the translations, over here?

@coenm
Copy link
Contributor Author

coenm commented Nov 5, 2020

If you want, I can provide some Dutch translations as well.

@@ -38,6 +38,7 @@ Alternativ, kann der Computer auch über das Menü oben rechts nach Repositories
<system:String x:Key="Open in Git Bash">In Git Bash öffnen</system:String>
<system:String x:Key="Open in Finder">In Finder öffnen</system:String>
<system:String x:Key="Open in Terminal">Im Terminal öffnen</system:String>
<system:String x:Key="Open in Sourcetree">Im Sourcetree öffnen</system:String>
Copy link
Owner

Choose a reason for hiding this comment

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

Sorry to say but in German, we'd say "In" instead of "Im". I know, German is hard. That's because ... it actually is.

Copy link
Owner

Choose a reason for hiding this comment

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

By the way, it is okay for "Im Terminal" 🙃

@awaescher
Copy link
Owner

By the way, I'd love to get Dutch translations if you don't mind.

@coenm
Copy link
Contributor Author

coenm commented Nov 6, 2020

By the way, I'd love to get Dutch translations if you don't mind.

I don't mind at all. I'll create a new pull request probably somewhere next week for the Dutch translations.

@awaescher awaescher merged commit 7100b58 into awaescher:master Nov 6, 2020
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