build: fix commit linter on unrebased PRs#39123
Conversation
The commit linter was checking out the PR HEAD commit instead of merge/rebase commit, causing it to fail for any PRs that were not rebased on our default branch.
mmarchini
left a comment
There was a problem hiding this comment.
If we add more dependencies it means we need to fetch more things, plus this PR unnecessarily increases the complexity of the Action.
I'm confused, this PR doesn't add a dependency, what do you mean by that? |
|
Dependencies in "external" files (as in files that are not the Action file, not actual dependencies). So if we move |
|
I agree that |
Alternative to #39121.
The commit linter was checking out the PR HEAD commit instead of
merge/rebase commit, causing it to fail for any PRs that were not
rebased on our default branch.