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.

Sign up for GitHub

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

Jump to bottom

adds Ruff lint workflow #308

Merged
merged 11 commits into from
Aug 8, 2024
Merged

adds Ruff lint workflow #308

merged 11 commits into from
Aug 8, 2024

Conversation

SpicyGarlicAlbacoreRoll
Copy link
Contributor

@SpicyGarlicAlbacoreRoll SpicyGarlicAlbacoreRoll commented Jul 13, 2024 •
edited
Loading

  • Adds ruff linting/formatting workflow (https://docs.astral.sh/ruff/)
    • Entire project linted and formatted
    • custom rules:
      • 79->100 character line limit
      • single quote string literals
    • Improves general readability of search method docstrings
  • Bumps release to major (v8.0.0)

@SpicyGarlicAlbacoreRoll SpicyGarlicAlbacoreRoll changed the title adds flake8 lint workflow adds Ruff lint workflow Aug 2, 2024
@SpicyGarlicAlbacoreRoll SpicyGarlicAlbacoreRoll merged commit b9473fa into master Aug 8, 2024
2 of 3 checks passed
@SpicyGarlicAlbacoreRoll SpicyGarlicAlbacoreRoll deleted the topic-flake8-action branch August 8, 2024 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@williamh890 williamh890 williamh890 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SpicyGarlicAlbacoreRoll @williamh890