PyWavelets / pywt

PyWavelets - Wavelet Transforms in Python
http://pywavelets.readthedocs.org
MIT License
1.97k stars 460 forks source link

Keep GitHub Actions up to date with GitHub's Dependabot #708

Closed cclauss closed 3 months ago

cclauss commented 3 months ago

Dependabot will fix the warnings are the bottom right of https://github.com/PyWavelets/pywt/actions/runs/8202874983

One instance is that the following should all be actions/setup-python@v5 https://github.com/search?q=repo%3APyWavelets%2Fpywt%20actions%2Fsetup-python&type=code

rgommers commented 3 months ago

Hi @cclauss, thank you for this PR, however I'm not really interested in adding Dependabot. It has pretty bad habits and a history of spamming open source projects. It's also overkill for a very slow-moving repo like this one. Having weekly or even monthly bot commits would mean ~50% of all PRs being from bots, and that's usually a bad sign and too much churn.

cclauss commented 3 months ago

Patterns * means that there can only be one pull request open at a time. GitHub Actions rarely receive major upgrades. Monthly means a theoretical maximum of 12 PRs a year (only one at any given time) and probably much less.

agriyakhetarpal commented 3 months ago

Hi @cclauss, I was adding this too in the meanwhile on a working branch of mine, when I happened to notice your PR – @rgommers, I would suggest getting this in so that we can have updates for GitHub Actions, now that we will be pushing nightly wheels soon (#706, https://github.com/scientific-python/upload-nightly-action/issues/75).

I would be happy to be assigned for review on such PRs. We could enable the rebase option as well in case that can help keep a single PR clean.

P.S. I was initially adding it on a daily cadence (I like that personally), but having it monthly would be perfectly fine.

cclauss commented 3 months ago

Generated: