-
Notifications
You must be signed in to change notification settings - Fork 1
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
93 synchronize linting setup between gh actions and local development #94
93 synchronize linting setup between gh actions and local development #94
Conversation
.github/workflows/linters.yml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
think we might also need to do the same with the toml file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated toml to remove the ^ for the ruff version
pyproject.toml
Outdated
@@ -21,7 +21,7 @@ alembic = "^1.12.0" | |||
pydantic-settings = "^2.0.3" | |||
|
|||
[tool.poetry.group.dev.dependencies] | |||
ruff = "^0.0.278" | |||
ruff = "0.0.278" | |||
black = "^23.7.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing black?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Closes #93
Updated the actions for black and ruff to have versions. Black can take a compatible version definition using ~=. However ruff-action has a strict regex that only allows a specific version to be defined, no concept of keeping the latest compatible major version.