search

redbrick / blockbot

Discord bot maintained by the Redbrick webgroup
GNU General Public License v3.0
1 stars 8 forks source link

Add Ruff to pre-commit #40

Closed ScuffedNewt closed 8 months ago

ScuffedNewt commented 8 months ago

Adds Ruff (mentioned in issue #31). This does not add nox as mentioned in issue 31 as nox is for "for automated testing" as mentioned on the issue so out of this PR's scope uses pre-commit

ScuffedNewt commented 8 months ago

I would like to mention that black and ruff have of course been run on this commit(s)

JedHazaymeh commented 8 months ago

I would like to mention that black and ruff have of course been run on this commit(s)

I noticed 😆 I am judging the styling. Are the current rules the default (and/or the recommended standard)?

ScuffedNewt commented 8 months ago

I noticed 😆 I am judging the styling. Are the current rules the default (and/or the recommended standard)?

yes current linting is by PEP 8 standards and default, no configs added etc

ScuffedNewt commented 8 months ago

removed black after some brief discussion on discord ðŸ¤