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

conventional commits for docs don't seem to be working #218

Open
barriebyron opened this issue Aug 25, 2022 · 2 comments
Open

conventional commits for docs don't seem to be working #218

barriebyron opened this issue Aug 25, 2022 · 2 comments
Labels

Comments

@barriebyron
Copy link
Contributor

Let's test the conventional commits rules in CI

I am not able to successfully use the following commit messages, which I think should be valid:

git commit -m "fix(docs): edits for whitepaper consistency, vocabulary, grammar"
git commit -m “fix(docs): update technical setup for first use”
git commit -m “feat(docs): add glossary list of terms”

Reach out to me and I'll help test when we are ready to verify the CI integration for conventional commits in regard to documentation commits.

@barriebyron
Copy link
Contributor Author

barriebyron commented Sep 6, 2022

@noandrea this is what @glebiller and I witnessed (saw it again today!), I am now working around the issue by using the --no-verify flag

➜  elesto git:(add-code-of-conduct-1) ✗ git commit -m "fix(docs): remove CoC from contributing because now isolated in CoC file"
[INFO] Initializing environment for https://github.com/golangci/golangci-lint.
[INFO] Installing environment for https://github.com/golangci/golangci-lint.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
golangci-lint........................................(no files to check)Skipped
golangci-lint........................................(no files to check)Skipped
Conventional Commit......................................................Failed
- hook id: conventional-pre-commit
- exit code: 1

grep: invalid option -- P
usage: grep [-abcdDEFGHhIiJLlMmnOopqRSsUVvwXxZz] [-A num] [-B num] [-C[num]]
	[-e pattern] [-f file] [--binary-files=value] [--color=when]
	[--context[=num]] [--directories=action] [--label] [--line-buffered]
	[--null] [pattern] [file ...]
[Commit message] fix(docs): remove CoC from contributing because now isolated in CoC file

Your commit message does not follow Conventional Commits formatting
https://www.conventionalcommits.org/

Conventional Commits start with one of the below types, followed by a colon,
followed by the commit message:

    build chore ci docs feat fix perf refactor revert style test

Example commit message adding a feature:

    feat: implement new API

Example commit message fixing an issue:

    fix: remove infinite loop

Optionally, include a scope in parentheses after the type for more context:

    fix(account): remove infinite loop

@github-actions
Copy link

This issue was marked as stale because there was no activity for 15 days

@github-actions github-actions bot added the Stale label Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant