ci: switch pre-commit action to bot

[Borda]

@jarrodmillman I wonder if there is a reason why using local GH action for lining and not the free bot - https://github.com/marketplace/pre-commit-ci but with this bot, you can also get automated fixes for lint
But before eventual merging, the bot needs to be installed so there is no degradation of code quality :)
Also, you can add a badge (the link will be available also upon installation)

[Borda]

Suggested change

	autoupdate_schedule: quarterly
	autoupdate_schedule: monthly

to be aligned with dependabot - https://github.com/scientific-python/lazy_loader/blob/2031223913c7e592c1c59aec72f296e9d08171fa/.github/dependabot.yml#L12

[jarrodmillman]

Thanks!! I will review these next week. I try to keep several different repos linting/ci stuff in sync so I can automate these kinds of linting / CI changes at the same time. So I will update the other repos at the same time.

[Borda]

@jarrodmillman, did you have a chance to have a look at this PR? 🦩

[jarrodmillman]

I am busy working on a grant proposal and a progress report for another grant that are due next week. I will take care of your PRs after that. I know they shouldn't take much time, but I am just swamped at the moment.

[Borda]

@jarrodmillman no worries 🎏 just let me know when you could check it... 🐿️

[Borda]

@jarrodmillman no worries 🎏 just let me know when you could check it... 🐿️

mind have a look, when you have time 🦩

[Borda]

@jarrodmillman no worries 🎏 just let me know when you could check it... 🐿️

mind have a look, when you have time 🦩

hello, any chance you may have some time to check this PR?

[stefanv]

Thanks for the PR, @Borda. I wonder if it's necessary to use the bot? We already get autofixes when we mention the bot by name: pre-commit.ci run

[Borda]

Thanks for the PR, @Borda. I wonder if it's necessary to use the bot? We already get autofixes when we mention the bot by name: pre-commit.ci run

interesting, did not know it, but does it also work for forked PRs?
(from my experience GH security distinguishes what is allowed for primary repo users and forked contributions)
if this command works always, then you are right and this does not give much advantage... :)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.74%. Comparing base (4e78314) to head (66a3af5).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #114   +/-   ##
=======================================
  Coverage   95.74%   95.74%           
=======================================
  Files           5        5           
  Lines         235      235           
=======================================
  Hits          225      225           
  Misses         10       10           

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

[stefanv]

Yes, as long as you don't need tokens, then you can run the action on PRs (take a look, e.g., at the other PRs in this repo). You can't rewrite them, but the bot can be manually invoked in those instances.

Anyway, appreciate you investigating CI improvements :)

[Borda]

seems it was merged with concurrent PR 😕 394af12