pre-commit: fix deprecated stages

[Crocmagnon]

Context

Fix deprecated pre-commit stages.
Fix #980

Validation

• Create a minimal pre-commit config (see below)
• Run pre-commit run --all-files
• No warning should appear.

Minimal config:

repos:
  - repo: https://github.com/gitguardian/ggshield
    rev: v1.34.0 # update this version
    hooks:
      - id: ggshield

PR check list

• As much as possible, the changes include tests (unit and/or functional)
• If the changes affect the end user (new feature, behavior change, bug fix) then the PR has a changelog entry (see doc/dev/getting-started.md). If the changes do not affect the end user, then the skip-changelog label has been added to the PR.

❓ Does this require a changelog entry? I'd be tempted to say no but I don't know your policies.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.03%. Comparing base (ee56436) to head (1558993).
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1029   +/-   ##
=======================================
  Coverage   92.03%   92.03%           
=======================================
  Files         181      181           
  Lines        7708     7708           
=======================================
  Hits         7094     7094           
  Misses        614      614           

Flag	Coverage Δ
unittests	92.03% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sevbch]

Hi @Crocmagnon ,
Thank you for this PR. To reproduce the validation case, could tell which version of pre-commit you're using?
Thanks!

[Crocmagnon]

version 4.0.1

[agateau-gg]

Thanks! I finally managed to reproduce and I can confirm your changes fit the warning.

Looking at a similar fix done by the pre-commit authors themselves, it looks like it is recommended to add a minimum_pre_commit_version: 3.2.0 entry for each hook. Can you look into this?

[agateau-gg]

Looks good, thanks!

[Crocmagnon]

Thank you for your quick feedback! :)