Skip to content
This repository has been archived by the owner on Sep 6, 2023. It is now read-only.

Add conventional commits #17

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

shairez
Copy link

@shairez shairez commented Sep 6, 2020

Following #8

I've added commitizen, conventional-changelog and commitlint (with a commit husky hook) to make sure the future commits follow the convention.

Plus added a fix for eslint on windows machines

now instead of calling "git commit" you need to run "yarn commit" instead. | This will run the
commitizen cli and let you format your commit messages for better changelogs for future versions | I
also added a pre-commit hook to lint your commits in order to prevent direct calls to "git commit"
that don't follow the convention

closes testimio#8
on windows I get a lot of "Expected linebreaks to be 'LF' but found 'CRLF'" errors before I git push
@Bnaya
Copy link
Contributor

Bnaya commented Sep 6, 2020

Hey Shai
Thanks for your PR!
We won't be able to accept this kind of change without internal discussion and RFC process

@shairez
Copy link
Author

shairez commented Sep 6, 2020

Sure thing

It didn't take long to do, so feel free to reject it if needed 😄

@benjamingr
Copy link
Contributor

benjamingr commented Sep 22, 2020

Hey, the CLA is live.

Shai can you please sign it? (saying "I agree to the CLA" in a GitHub comment is sufficient for now assuming your GitHub account and email are linked) - It transfers your kidneys ownership of committed code to us after you commit so we can consume the code with a different license internally.

@shairez
Copy link
Author

shairez commented Sep 22, 2020

"I agree to the CLA" 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants