neurobagel / workflows

MIT License
0 stars 1 forks source link

pre-commit-ci fails when automatically created CHANGELOG PRs are merged #87

Closed alyssadai closed 1 month ago

alyssadai commented 4 months ago

In repos where auto is set up for auto-releasing and pre-commit-ci is enabled, when a release is triggered (by a PR merge), another PR is automatically created which updates the changelog. This PR is then merged automatically, meaning pre-commit-ci does not have time to run on it. We think this is what's causing it to fail upon the merge to main (e.g., https://results.pre-commit.ci/run/github/567874792/1713298808.705SJF3VSvec-P8euGQopw).

One potential solution is to configure the auto-release workflow to include in the commit message for the changelog something like the following (from pre-commit-ci logs) to skip pre-commit CI:

skipping push runs: skip a run by putting [skip ci], [ci skip], [skip pre-commit.ci], or [pre-commit.ci skip] in the commit message.

See also: https://intuit.github.io/auto/docs/generated/changelog

alyssadai commented 1 month ago

Re-opening as adding single additional skip keyword [skip pre-commit.ci] did not do the trick.