Skip to content

Commit

Permalink
fix(ruff): use the same command for the ci for python checks
Browse files Browse the repository at this point in the history
CHANGELOG:
- [ ] Use consistent command for ruff check throughout.
- [ ] Don't install node as prerequisite since nvm installs it.
  • Loading branch information
yesudeep committed Feb 18, 2025
1 parent d3261b8 commit 817c7b5
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ jobs:
run: uv run --directory py ruff format --check .

- name: Lint with ruff
run: uv run --directory py ruff check .
run: uv run --directory py ruff check --select I .

- name: Check licenses
run: ./bin/check_license
Expand Down
2 changes: 0 additions & 2 deletions bin/setup
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,6 @@ function genkit::install_prerequisites() {
curl \
fd-find \
gh \
nodejs \
python3 \
ripgrep
elif [[ -x "$(command -v dnf)" ]]; then
Expand All @@ -136,7 +135,6 @@ function genkit::install_prerequisites() {
fd-find \
gh \
go \
node \
python3 \
ripgrep
else
Expand Down
2 changes: 1 addition & 1 deletion py/bin/generate_schema_types
Original file line number Diff line number Diff line change
Expand Up @@ -21,4 +21,4 @@ python3 "${TOP_DIR}/py/bin/sanitize_schema_types.py" "${SCHEMA_FILE}"
uv run --directory "${TOP_DIR}/py" \
ruff format "${TOP_DIR}"
uv run --directory "${TOP_DIR}/py" \
ruff check --fix "${SCHEMA_FILE}"
ruff check --select I --fix "${SCHEMA_FILE}"

0 comments on commit 817c7b5

Please sign in to comment.