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

ci(commitlint): addition of commit message linter #191

Merged
merged 2 commits into from
Jan 12, 2024

Conversation

tiborsimko
Copy link
Member

Adds commitlint to check the commit message style against agreed conventional commits configuration.

Changes script argument values to always use linter names (e.g. shellcheck).

Changes argument handling to allow only one checking action that can now accept further optional arguments.

Copy link

codecov bot commented Jan 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6767d33) 28.47% compared to head (d40a675) 28.47%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #191   +/-   ##
=======================================
  Coverage   28.47%   28.47%           
=======================================
  Files           5        5           
  Lines         144      144           
=======================================
  Hits           41       41           
  Misses        103      103           

tiborsimko added a commit to tiborsimko/reana-workflow-engine-serial that referenced this pull request Jan 10, 2024
Adds commitlint to check the commit message style against agreed
conventional commits configuration.

Changes script argument values to always use linter names (e.g.
shellcheck).

Changes argument handling to allow only one checking action that can now
accept further optional arguments.
tiborsimko added a commit to tiborsimko/reana-workflow-engine-serial that referenced this pull request Jan 10, 2024
MANIFEST.in Outdated
include *.rst
include *.sh
include *.txt
include *.yaml
include *.yaml
Copy link
Member

Choose a reason for hiding this comment

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

include *.yaml it's duplicated in MANIFEST.in

Adds commitlint to check the commit message style against agreed
conventional commits configuration.

Changes script argument values to always use linter names (e.g.
shellcheck).

Changes argument handling to allow only one checking action that can now
accept further optional arguments.
tiborsimko added a commit to tiborsimko/reana-workflow-engine-serial that referenced this pull request Jan 11, 2024
@tiborsimko tiborsimko merged commit d40a675 into reanahub:master Jan 12, 2024
14 checks passed
@tiborsimko tiborsimko deleted the commitlint branch January 12, 2024 11:19
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.

2 participants