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

Added plugin with jira rules for commitlint-config-jira #14

Merged

Conversation

jotanavarro
Copy link
Contributor

The plugin containing rules to be used by commitlint-config-jira was missing from the package.json. This means that when extending ['jira'] in the commitlint.config.js it was always failing due to unknown rules.

This fixes #13 and allows easy usage of the commitlint-config-jira.

Copy link
Owner

@wagoid wagoid left a comment

Choose a reason for hiding this comment

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

hi @jotanavarro, thank you for fixing this issue! The changes look good to me, there's just one thing: could you update your commit message to use the conventional commits pattern?
It would be something like this:

fix: added plugin with jira rules for `commitlint-config-jira`

Also updated the `commitlint-config-jira` to its latest version.

Sorry for CI not telling you that, the reason why our current tests didn't show lint errors in your message is because the action was always working on CI due to a bug in the workflow I introduced recently. I just fixed it here so if you sync with master again you will have a CI that is not "incorrectly passing"

@jotanavarro jotanavarro force-pushed the Adding-commitlint-jira-plugin-for-rules branch 3 times, most recently from 2a2f674 to 3bb0d67 Compare November 24, 2019 11:37
Added the `commitlint-plugin-jira-rules` package. Also updated
the `commitlint-config-jira` to its latest version.
@jotanavarro jotanavarro force-pushed the Adding-commitlint-jira-plugin-for-rules branch from 3bb0d67 to 6cede4b Compare November 24, 2019 11:43
@jotanavarro jotanavarro requested a review from wagoid November 24, 2019 11:45
@jotanavarro
Copy link
Contributor Author

Now using the more verbose commit message policy.

Copy link
Owner

@wagoid wagoid left a comment

Choose a reason for hiding this comment

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

🚀

@wagoid wagoid merged commit 526db9a into wagoid:master Nov 24, 2019
@jotanavarro jotanavarro deleted the Adding-commitlint-jira-plugin-for-rules branch November 24, 2019 13:25
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.

Commitlint for JIRA apparently not packed
2 participants