wagoid/commitlint-github-action

How should the commit message "Merge branch 'main' into main" be handled?

Axedyson opened this issue · 2 comments

Axedyson commented

Take a loot at the merge commits at the bottom of this PR. The commitlint action reports these as errors. How should such commits be handled, is there some preferred/best practice way?

jdbruijn commented

I think the commit messages like "Merge" and "Revert" are actually ignored, as they should. Please have a closer look at you'r action errors. The first failure on the Merge remote-tracking branch 'upstream/main' into main commit is actually on the lint workflow and all later commits look to fail because of the lol commit message of commit Axedyson/syncbase@226bbfd (for a PR we check all commits in the PR). Please reopen if that was not the issue, but from my quick analysis looks like it is.

  • 👍1
Axedyson commented

You're right!