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

compatibility with 'git commit -s' #178

Merged
merged 1 commit into from
Feb 16, 2021
Merged

Conversation

gsemet
Copy link
Contributor

@gsemet gsemet commented Feb 12, 2021

git 2.30: git commit -s adds a Signed-off-by line, not a Signed-Off-By.

@jorisroovers
Copy link
Owner

Thanks!

Would you mind fixing the failing tests? Appreciate it!

git 2.30: `git commit -s` adds a `Signed-off-by` line,
not a `Signed-Off-By`.

Signed-off-by: Gaetan Semet <gaetan.semet@renault.com>
@gsemet
Copy link
Contributor Author

gsemet commented Feb 12, 2021

done

@jorisroovers jorisroovers merged commit 9d87cc2 into jorisroovers:main Feb 16, 2021
@jorisroovers
Copy link
Owner

Thanks!

jorisroovers added a commit that referenced this pull request Apr 16, 2021
Bugfixes:
  - Git commit message body with only new lines is not longer considered empty
    by `body-is-missing` (#176)
  - Added compatibility with `git commit -s` for
    `contrib-requires-signed-off-by` rule (#178)
- Minor tweak to gitlint commit-hook output (#173)
- All dependencies have been upgraded to the latest available versions
- Minor doc fixes

Full Release details in CHANGELOG.md.
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