Repository navigation
Add an option to make messages capitalized #80
Description
Activity
Hi, thanks for the interest and suggestion.
I'll add this as something you can control in settings.
Hi, I added a draft PR. I got this working for a narrow usecase, now need to get it to work for multiple file changes and when not using a type prefix.
Here's how it is configured:
Hey that looks great! Sorry for late response.
I see that PR is failing on build but the logs are expired, so I cannot see them.
Could you please re-run the job?I have forked your version and fixed linter errors, however I was not able to build current version due to error
Error: Cannot find module 'vscode'https://github.com/avixFF/auto-commit-msg/actions/runs/3463893007/jobs/5784743972
Alright, so I figured out why there was build problem and it was not at all trivial.
Turns out Microsoft has updated vscode package.
This in turn required mocha downgrade to version 4
Which required rewriting tests using onlytest()function
That triggered linter, since it made lines greater than 100 characters long
Because of that I had to shorten some test messagesSo in order to use
vscodepackage you have to run tests inside an actual vscode instance. Document linked above explains it.
The build was failing because you were including prepareCommitMsg.ts in prepareCommitMsg.test.ts which hadvscodepackage import that was not accessible when run with mocha from CLI.In my branch I have changed that your mocha tests run inside vscode instance, therefore
vscodepackage is accessible for tests.Here is my version: lexuzieel@7440e07
While we are at it, I would also like to add a flag to disable prefix (I don't use those for my projects). I have created a separate PR based on the latest version of my feat-titlecase branch: #86
Thanks for the info. Reviewing your PR now
Hello!
In my repos I write messages starting with capital letter. Whilst this extension gets the message correct, it does not capitalize the message.
i.e. currently when I change a file and press a button, I get this message:
I have to change the first letter of the message to capital manually:
Is there a way to change this behavior (probably through settings)?