Skip to content

Commit fd20f5f

Browse files
amosttAygentic
andcommitted
fix(ci): properly configure uv environment for lint script
- Revert lint.sh to use direct commands (not uv run) since the script runs in uv environment - Add 'uv sync' step to lint-backend.yml to install dependencies before running - Commands in bash scripts run via 'uv run bash script.sh' already have access to the uv environment Per uv documentation, when running 'uv run bash scripts/lint.sh', the environment is already set up, so commands like mypy and ruff can be called directly without uv run prefix. Related to CUR-29 🤖 Generated by Aygentic Co-Authored-By: Aygentic <noreply@aygentic.com>
1 parent be68275 commit fd20f5f

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

.github/workflows/lint-backend.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,5 +24,9 @@ jobs:
2424
with:
2525
version: "0.4.15"
2626
enable-cache: true
27-
- run: uv run bash scripts/lint.sh
27+
- name: Install dependencies
28+
run: uv sync
29+
working-directory: backend
30+
- name: Run linters
31+
run: uv run bash scripts/lint.sh
2832
working-directory: backend

backend/scripts/lint.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@
33
set -e
44
set -x
55

6-
uv run mypy app
7-
uv run ruff check app
8-
uv run ruff format app --check
6+
mypy app
7+
ruff check app
8+
ruff format app --check

0 commit comments

Comments
 (0)