search

npc203 / npc-cogs

My list of Cogs for Red-bot :p
MIT License
25 stars 26 forks source link

Run pre-commit #70

Closed Zephyrkul closed 1 year ago

Zephyrkul commented 2 years ago

Nothing more than just using pre-commit run --all-files so that (hopefully, if I did it right) your main branch will be passing checks again.

I looked over the changed files and it looks to all be the same beyond formatting and whitespace changes, but I haven't actually installed and ran this repo following these changes so your mileage may vary.

Zephyrkul commented 2 years ago

aight looks like more will be needed but it's a simple fix, it'll require a change to your pre-commit yaml though

Zephyrkul commented 2 years ago

All set, checks are green again.

I'll note that the issue with black and click is better resolved by bumping the version of black that your precommit uses, but this PR is focused solely on making your checks all green for your main branch.

If you want me to bump your precommit's dependencies feel free to ask me to.

npc203 commented 1 year ago

Yup, sure, I'll merge after you bump the deps.

Zephyrkul commented 1 year ago

Should be all set.

npc203 commented 1 year ago

Thanks for the PR!