feat: detect default stash message microsoft#106907#107074
feat: detect default stash message microsoft#106907#107074joaomoreno merged 3 commits intomicrosoft:masterfrom dfireBird:feature-106907
Conversation
joaomoreno
left a comment
There was a problem hiding this comment.
I would put this behind a setting. The text in the commit box might be equal to the commit template, which would be a poor indicator of a good stash message. Optionally, we can compare the value against the last known template message and only use it if it doesn't equal the template message.
|
By setting do you mean, a setting in preferences to toggle whether to use commit message as default stash message? If so, can you give me a hint on how to do it? Sorry I never used git commit template, so I didn't knew about it. I can change the code. Edit-1: Updated question. |
|
Yes, a user setting that can be configured. When I am looking to add a setting, I search for a setting that I know exists. Then kind of follow what was needed to implement it. For example, you could search for “search.actionsPosition” in the src folder. Then you should be able to find it and see what sort of implementation you need. (It may be better to search for a git setting, so you’re in the same area already.) Somebody else may be able to give you better details about what exactly is required. I don’t work in it enough to know the specifics. |
I figured that out and already added in commit 94ab148 in my fork. |
use commit message as default stash message if commit message box is populated
|
@joaomoreno Is this PR still in review? |
This PR fixes #106907
Use commit message as default stash message if commit message box is populated.