Skip to content

Commit 689e125

Browse files
authored
Use the pinned version of ruff in CI (baserow#4651)
* Use the pinned version of ruff in CI * Use latest setup-uv * Use correct version, not tag version * Lint fix frontend
1 parent 6de2e41 commit 689e125

2 files changed

Lines changed: 19 additions & 7 deletions

File tree

.github/workflows/ci.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -206,15 +206,15 @@ jobs:
206206
- uses: actions/checkout@v4
207207

208208
- name: Install uv
209-
uses: astral-sh/setup-uv@v4
209+
uses: astral-sh/setup-uv@v7
210210
with:
211-
version: "0.5.6"
211+
version: "0.9.29"
212212

213213
- name: Run ruff
214214
working-directory: backend
215215
run: |
216-
uvx ruff check src/ ../premium/backend/src/ ../enterprise/backend/src/ tests/ ../premium/backend/tests/ ../enterprise/backend/tests/
217-
uvx ruff format --check src/ ../premium/backend/src/ ../enterprise/backend/src/ tests/ ../premium/backend/tests/ ../enterprise/backend/tests/
216+
uv run ruff check src/ ../premium/backend/src/ ../enterprise/backend/src/ tests/ ../premium/backend/tests/ ../enterprise/backend/tests/
217+
uv run ruff format --check src/ ../premium/backend/src/ ../enterprise/backend/src/ tests/ ../premium/backend/tests/ ../enterprise/backend/tests/
218218
219219
frontend-lint:
220220
name: Web-Frontend Lint

web-frontend/modules/core/components/sidebar/SidebarWithWorkspace.vue

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,19 @@
1616
</ul>
1717
<ul class="tree">
1818
<li
19-
v-for="(applicationGroup, index) in groupedApplicationsForSelectedWorkspace"
19+
v-for="(
20+
applicationGroup, index
21+
) in groupedApplicationsForSelectedWorkspace"
2022
:key="applicationGroup.type"
2123
>
22-
<div class="tree__heading" :class="{'margin-bottom-2': applicationGroup.applications.length === 0 && index < groupedApplicationsForSelectedWorkspace.length - 1}">
24+
<div
25+
class="tree__heading"
26+
:class="{
27+
'margin-bottom-2':
28+
applicationGroup.applications.length === 0 &&
29+
index < groupedApplicationsForSelectedWorkspace.length - 1,
30+
}"
31+
>
2332
<div class="tree__heading-name">
2433
{{ applicationGroup.name }}
2534
</div>
@@ -73,7 +82,10 @@
7382
>
7483
</component>
7584
</ul>
76-
<div v-if="index < groupedApplicationsForSelectedWorkspace.length - 1" class="tree__separator"></div>
85+
<div
86+
v-if="index < groupedApplicationsForSelectedWorkspace.length - 1"
87+
class="tree__separator"
88+
></div>
7789
</li>
7890
</ul>
7991
</div>

0 commit comments

Comments
 (0)