Skip to content

Commit c4f6281

Browse files
heysamtexasclaude
andcommitted
refactor: eliminate ruff duplication in GitHub Actions workflows
- Add workflow_call trigger to ruff.yml to make it reusable - Make release.yml depend on both test.yml and ruff.yml workflows - Remove duplicate ruff check and format steps from release.yml - Improves maintainability by centralizing linting logic in one place 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 2adf1a4 commit c4f6281

File tree

2 files changed

+5
-8
lines changed

2 files changed

+5
-8
lines changed

.github/workflows/release.yml

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,12 @@ jobs:
2121
test:
2222
uses: ./.github/workflows/test.yml
2323

24+
ruff:
25+
uses: ./.github/workflows/ruff.yml
26+
2427
release:
2528
runs-on: ubuntu-latest
26-
needs: test
29+
needs: [test, ruff]
2730
steps:
2831
- name: Checkout repository
2932
uses: actions/checkout@v4
@@ -44,12 +47,6 @@ jobs:
4447
- name: Install dependencies
4548
run: uv sync --extra dev
4649

47-
- name: Run linting
48-
run: uv run ruff check .
49-
50-
- name: Run formatting check
51-
run: uv run ruff format --check .
52-
5350
- name: Configure Git
5451
run: |
5552
git config user.name "github-actions[bot]"

.github/workflows/ruff.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
name: Ruff
2-
on: [push, pull_request]
2+
on: [push, pull_request, workflow_call]
33
permissions:
44
contents: read
55
jobs:

0 commit comments

Comments
 (0)