Adopt ruff for python linting in CI/CD #1149
Labels
enhancement
New feature or request
good first issue
Good for newcomers
help wanted
Extra attention is needed
Comments
roeap
added
enhancement
|
Yeah, I have been thinking of adding switching to ruff as well! |
|
Good idea 👍 |
wjones127
pushed a commit
that referenced
this issue
Feb 22, 2023
# Description This PR sets up ruff for python linting. It also enables isort from within ruff and removes the standalone isort. # Related Issue(s) closes #1149 --------- Signed-off-by: Marijn Valk <marijncv@hotmail.com>
|
For reference, I created a PR on the ruff repository to add delta-rs to the list of users |
chitralverma
pushed a commit
to chitralverma/delta-rs
that referenced
this issue
Mar 17, 2023
# Description This PR sets up ruff for python linting. It also enables isort from within ruff and removes the standalone isort. # Related Issue(s) closes delta-io#1149 --------- Signed-off-by: Marijn Valk <marijncv@hotmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
As proud members of the Rust community, as well as advocates of rust backed python, I thought we could adopt ruff as our main python linter in CI/CD.
Its not a huge pain right now, but the change is also quite minimal in terms of effort, and I personally made great experiences with it several projects.
As a bonus we could add delta-rs to the list of adopters alongside many other great projects like, pydantic, polars, pandas and many more...
Use Case
Related Issue(s)
The text was updated successfully, but these errors were encountered: