Skip to content

Lint sh cleanup #434

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

Merged
merged 2 commits into from
Apr 25, 2025
Merged

Lint sh cleanup #434

merged 2 commits into from
Apr 25, 2025

Conversation

cdorsman
Copy link

Cleaned up lint.sh:

  • Removed first execution of Black because Black is already executed by isort.
  • Specifically directed all the linting tools to the specific location of ./patterns. By not doing so, users will experience much more noise from the linting tools when directories like venv/ and .tox/ and setup.py located in the root are being linted.
  • Moved all installs of packages on top of the file, so that the rest of the output is not getting trashed by it.

@faif faif merged commit ac1b2b4 into faif:master Apr 25, 2025
1 check passed
@cdorsman cdorsman deleted the lint-sh-cleanup branch April 25, 2025 15:04
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