ci: update pre-commit hooks, fix linting, and refresh dependencies#1385
Open
dariocurr wants to merge 7 commits intoapache:mainfrom
Open
ci: update pre-commit hooks, fix linting, and refresh dependencies#1385dariocurr wants to merge 7 commits intoapache:mainfrom
dariocurr wants to merge 7 commits intoapache:mainfrom
Conversation
* Update Ruff version in pre-commit configuration to v0.15.1. * Add noqa comments to suppress specific linting warnings in various files. * Update regex patterns in test cases for better matching.
* Adjusted indentation for the enable-cache option in the test.yml workflow file to ensure proper YAML formatting.
* Adjusted the order of imports in indexed_field.rs to improve readability and maintain consistency with project conventions.
* Bump versions of several dependencies including tokio, pyo3-log, prost, uuid, and log to their latest releases. * Update Cargo.lock to reflect the changes in dependency versions.
* Adjusted formatting in pyproject.toml for improved readability by aligning lists and ensuring consistent indentation. * Updated dependencies and configuration settings for better organization.
* Cleaned up import statements in multiple files by removing unnecessary noqa comments, enhancing code readability and maintaining consistency across the codebase.
* Streamlined list formatting in pyproject.toml for improved readability by removing unnecessary line breaks and ensuring consistent structure across sections. * No functional changes were made; the focus was solely on code style and organization.
Member
|
Thanks for the PR. I see CI is passing now. We have the release vote in process, so I'm going to hold off on review for a couple of days, just until the release merges in. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Which issue does this PR close?
N/A — no issue. This PR is tooling/CI and formatting only (pre-commit, lint fixes, dependency bumps).
Rationale for this change
Keep the repo aligned with current pre-commit hooks and linters, fix resulting style/format issues, and refresh build dependencies so CI and local checks stay consistent and up to date.
What changes are included in this PR?
.pre-commit-config.yamland fix indentation in.github/workflows/test.yml.pyproject.toml,dataframe.py,expr.py,plan.py,user_defined.py,input/location.py, and test files inpython/tests/).src/expr/indexed_field.rsfor clarity.Cargo.tomlandCargo.lock.No behavior or API changes.
Are there any user-facing changes?
No. Changes are limited to tooling, CI, formatting, and dependency versions.