Skip to content

Conversation

@callumforrester
Copy link
Contributor

Fixes #282

Enable pep8-naming ruff rules, enforcing that projects follow convetional pep8 naming styles. See https://peps.python.org/pep-0008

@callumforrester callumforrester changed the title Enable pep8-naming ruff rules feat: Enable pep8-naming ruff rules Aug 11, 2025
Fixes #282

Enable pep8-naming ruff rules, enforcing that projects follow convetional pep8 naming styles. See https://peps.python.org/pep-0008
@DiamondJoseph
Copy link
Contributor

DiamondJoseph commented Aug 11, 2025

Isn't this already done and earlier by the Jinja answer verification?

{% if not (package_name | regex_search('^[a-zA-Z][a-zA-Z_0-9]+$')) %}

e: spoken IRL, I thought this was just chekcing hat the name of the project was compliant, not that naming within the project was compliant. With that in mind, LGTM

@DiamondJoseph DiamondJoseph merged commit 9d544a5 into main Aug 11, 2025
6 checks passed
@DiamondJoseph DiamondJoseph deleted the 282-pep8-ruff branch August 11, 2025 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable ruff pep8 naming rules

4 participants