Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Python precommits and other goodies #58

Merged
merged 3 commits into from
Oct 16, 2024

Conversation

wtfzambo
Copy link
Collaborator

@wtfzambo wtfzambo commented Oct 16, 2024

In this PR:

  • Use Ruff only for Python linting and formatting (disable Black)
  • Add isort linting rules
  • Add pyfix script to quickly run ruff checker and formatter

Also:

  • Add show script to base template: prints all available commands with descriptions
  • Improve jinja block formatting
  • Bundle all scripts under single script object to improve readability
  • Keep .envrc files in new repos (use .env to set local env variables).

Federico Zambelli added 3 commits October 16, 2024 10:42
Use `.env` to set local env variables
- Use Ruff as linter and formatter
- Add isort to linting rules
- Add script to base template to print available commands
- Improve jinja block formatting
- Bundle all scripts under single `script` object for readability
@wtfzambo wtfzambo requested a review from shahinism October 16, 2024 09:18
@shahinism
Copy link
Collaborator

Good job @wtfzambo!

@shahinism shahinism merged commit c08a962 into main Oct 16, 2024
2 checks passed
@shahinism shahinism deleted the feat/py-precommits-and-other-stuff branch October 16, 2024 09:33
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.

2 participants