Wagner Santos
Wagner Santos
Hey @sonrohan! That seems like a nice feature. @jdbruijn your comment made me wonder if we should support a similar `fetch-depth` param in the action? So if we pass `fetch-depth:...
Hi @michaelgwelch! Basically we run a command similar to this to get all commit messages of a PR: `git log --first-parent [fromHash]^1..[toHash]`. In theory, this should be enough to be...
@michaelgwelch got it, thanks for setting up the repo! Another thing you can try to fix this quicker is to rebase one branch into the other instead of merging. Maybe...
I've created a github action for this that doesn't require any setup, if you want to have a look: https://github.com/marketplace/actions/commit-linter I tried using @bennypowers's action but got some issues and...
Hi @Sakethtadimeti, the issue is still open. I won't be able to work on it in the near future, but happy to review any PRs!
@Sakethtadimeti you will need to create a fork of the repo, then submit a pull request.
hi @michaelgwelch! TBH I don't remember anymore why I don't just use the result from github API. Maybe at the time I thought commit names were not in the response....
Nice catch! Thanks for identifying this, will have a look later
Hey everyone! Sorry for the delay on this, I've just released a new major version that changes the behavior to use github API. You don't need `fetch-depth: 0` anymore 💪
Correction: there was an issue with the release, it got released as a patch. Anyway it should be fine, there's only `firstParent` input that I removed, shouldn't affect most users.