auto-commit-msg

auto-commit-msg copied to clipboard

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)?

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 only test() function That triggered linter, since it made lines greater than 100 characters long Because of that I had to shorten some test messages

So in order to use vscode package 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 had vscode package 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 vscode package is accessible for tests.

Here is my version: https://github.com/avixFF/auto-commit-msg/commit/7440e072b61beaa3804f258c57037a249282d575

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