Skip to content

fix(pre-commit): remove deprecated stage name & port to GitHub Actions#5

Merged
iainsproat merged 2 commits intomainfrom
iain/update-pre-commit-stage
Apr 3, 2026
Merged

fix(pre-commit): remove deprecated stage name & port to GitHub Actions#5
iainsproat merged 2 commits intomainfrom
iain/update-pre-commit-stage

Conversation

@iainsproat
Copy link
Copy Markdown
Contributor

  • pre-commit has deprecated the 'commit' stage name, have replaced with 'pre-commit'
  • chore: bumps pre-commit config dependencies
  • provides better message in ggshield hook when CI is detected
  • replaces CircleCI with GitHub Actions

Description & motivation

Changes:

To-do before merge:

Screenshots:

Validation of changes:

Checklist:

  • My pull request follows the guidelines in the Contributing guide?
  • My pull request does not duplicate any other open Pull Requests for the same update/change?
  • My commits are related to the pull request and do not amend unrelated code or documentation.
  • My code follows a similar style to existing code.
  • I have added appropriate tests.
  • I have updated or added relevant documentation.

References

- pre-commit has deprecated the 'commit' stage name, have replaced with 'pre-commit'
- chore: bumps pre-commit config dependencies
- provides better message in ggshield hook when CI is detected
@iainsproat iainsproat changed the title fix(pre-commit): remove deprecate stage name fix(pre-commit): remove deprecated stage name & port to GitHub Actions Apr 3, 2026
@iainsproat iainsproat marked this pull request as ready for review April 3, 2026 09:31
@iainsproat iainsproat merged commit e4bde6c into main Apr 3, 2026
2 checks passed
@iainsproat iainsproat deleted the iain/update-pre-commit-stage branch April 3, 2026 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant