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

Add linting and formatting tools configuration #194

Merged
merged 4 commits into from
Sep 19, 2024
Merged

Add linting and formatting tools configuration #194

merged 4 commits into from
Sep 19, 2024

Conversation

meanmail
Copy link
Contributor

Added ruff and mypy configurations in pyproject.toml to enable strict linting and static type checking. This includes specifications for excluded directories, line length, complexity limits, inline quotes style, and various mypy checks to ensure code quality and consistency. Additionally, moved jsonschema to correct dependency section and enhanced pytest setup with additional options.

Issues:
Issue#

Description:

Added `ruff` and `mypy` configurations in `pyproject.toml` to enable strict linting and static type checking. This includes specifications for excluded directories, line length, complexity limits, inline quotes style, and various mypy checks to ensure code quality and consistency. Additionally, moved `jsonschema` to correct dependency section and enhanced pytest setup with additional options.
@meanmail meanmail self-assigned this Sep 19, 2024
meanmail added 3 commits September 19, 2024 19:50
Replace flake8 with ruff in the GitHub Actions build workflow. The new tool improves linting checks and applies fixes automatically where possible.
Added 'hyperstyle/src/python/review/inspectors/common/inspector/proto' to the exclusion lists in two different configuration sections to ensure the directory is ignored by relevant tools. This change will help maintain cleaner project management and avoid unnecessary processing of proto files.
@meanmail meanmail merged commit 0d742a8 into main Sep 19, 2024
4 checks passed
@meanmail meanmail deleted the ruff branch September 19, 2024 18:24
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.

1 participant