ci: use pre-commit also locally

[Borda]

Checklist before merging this PR:

• Mentioned all issues that this PR fixes or addresses.
• Summarized the updates of this PR under Summary.
• Added an entry under Unreleased in the Changelog.

Addresses #2323 (comment)

Summary

use pre-commit also locally instead of calling separate tools

Other Information

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.01%. Comparing base (8f8b514) to head (c0f76d9).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2327   +/-   ##
=======================================
  Coverage   94.01%   94.01%           
=======================================
  Files         138      138           
  Lines       14187    14187           
=======================================
  Hits        13338    13338           
  Misses        849      849           

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

[madtoinou]

Thanks @Borda for creating a separate PR for this, using pre-commit as a command for linting/formatting looks very neat.

[dennisbader]

Looks great @Borda, thanks for this contribution 🚀

Could you add an entry to CHANGELOG.md under this section (for the other PRs as well)?

Then we're ready to merge!

[Borda]

Could you add an entry to CHANGELOG.md under this section (for the other PRs as well)?

added 53256bc

@madtoinou @dennisbader so ready to land? 🦩

[dennisbader]

Hi @Borda, there are merge conflicts from the CHANGELOG.md. I could fix these myself for all the PRs however, it seems I don't have write permission to the forked repo.
Would it be possible to grant me access (I'd also like to slightly adapt the content for the changes in CHANGELOG)?
If not, you can also fix the conflicts, we merge all PRs and I'll update the content later.

[Borda]

Would it be possible to grant me access (I'd also like to slightly adapt the content for the changes in CHANGELOG)?

I di not do any restrictions, so all maintainers shall be able to push

[dennisbader]

@Borda, the other PR was merged. If you merge the master into this one and move the changelog entry to the unreleased section, we're good to go.

[Borda]

@dennisbader ready