search

catalyst-cooperative / pudl-catalog

An Intake catalog for distributing open energy system data liberated by Catalyst Cooperative.
https://catalyst.coop/pudl/
MIT License
9 stars 2 forks source link

[pre-commit.ci] pre-commit autoupdate #85

Closed pre-commit-ci[bot] closed 1 year ago

pre-commit-ci[bot] commented 1 year ago

updates:

jdangerx commented 1 year ago

Welcome back @zaneselvans ! I noticed that most of the recent commits in this repo are either pre-commit updates or attempting to disable pre-commit.ci bot PRs. Should we keep this open? Slash should we just remove the pre-commit hook completely for now?

zaneselvans commented 1 year ago

@jdangerx right now it's an infrequently updated repo, but it should probably become an automatically updated repo that produces a new catalog version whenever we create a PUDL data release. I was trying to disable those auto PRs because at the time it was pulling a bunch of real data and causing egress fees. Which I think we've now fixed. (right @bendnorman ?).

But anyway the catalog is broken / not ready for use right now so maybe it does make sense to disable these auto PRs for now and just tackle it all when we revisit auto-updating catalogs.

jdangerx commented 1 year ago

Ha - I was confused why commenting everything out didn't stop the PRs - it turns out that

note: ci: configuration is entirely optional and will assume the defaults below.

(docs)

It also appears that autoupdate_schedule can't be set to "never" and there seems to be no way to turn it off. So I'm going to close this and open a PR that sets autoupdate_schedule to "quarterly" so at least we only have to close 1 PR a quarter until we're ready to deal with this.