Skip to content

gitlint: ignore dependabot Co-authored-by lines #373

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

Merged
merged 1 commit into from
Nov 24, 2022

Conversation

jorisroovers
Copy link
Owner

Dependabot tends to generate "Co-authored-by" lines that exceed the default
80 char limit. Ignore those lines in commit messages.

@jorisroovers jorisroovers self-assigned this Nov 21, 2022
@jorisroovers
Copy link
Owner Author

Need to merge #374 first and then rebase this

@jorisroovers jorisroovers added this to the 0.19.0 milestone Nov 21, 2022
@webknjaz
Copy link
Contributor

Talking about Co-Authored-By:, it could be useful for M1: author-valid-email to also check those trailers and the committer? You can extract these via something like git show --no-patch --pretty='format:%(trailers:key=Co-Authored-By,separator=%x0A)%x0A%an <%ae>%x0A%cn <%ce>' HEAD.

Dependabot tends to generate "Co-authored-by" lines that exceed the default
80 char limit. Ignore those lines in commit messages.
@jorisroovers jorisroovers force-pushed the gitlint-ignore-dependabot-co-authored-by branch from 4112b71 to 5a3d173 Compare November 24, 2022 09:14
@jorisroovers jorisroovers merged commit 23bedf1 into main Nov 24, 2022
@jorisroovers jorisroovers deleted the gitlint-ignore-dependabot-co-authored-by branch January 23, 2023 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 0.19.0
Development

Successfully merging this pull request may close these issues.

2 participants