Skip to content

Commit e54973b

Browse files
committed
Remove claude review action, move linting to another action
1 parent ee2a845 commit e54973b

File tree

5 files changed

+46
-87
lines changed

5 files changed

+46
-87
lines changed

.github/workflows/claude.yml

Lines changed: 0 additions & 64 deletions
This file was deleted.

.github/workflows/lint.yml

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
name: Lint
2+
3+
on:
4+
push:
5+
branches: [main]
6+
pull_request:
7+
branches: [main]
8+
9+
jobs:
10+
lint:
11+
runs-on: ubuntu-latest
12+
strategy:
13+
matrix:
14+
python-version: ["3.11"]
15+
16+
steps:
17+
- uses: actions/checkout@v3
18+
- name: Set up Python ${{ matrix.python-version }}
19+
uses: actions/setup-python@v4
20+
with:
21+
python-version: ${{ matrix.python-version }}
22+
- name: Install dependencies
23+
run: |
24+
python -m pip install --upgrade pip
25+
pip install ruff
26+
- name: Lint with ruff
27+
run: |
28+
ruff check .
29+
- name: Format check with ruff
30+
run: |
31+
ruff format --check .

.github/workflows/testprimitives.yml

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,15 @@
11
name: Primitives
22
on:
3-
push:
4-
branches: [main]
5-
pull_request:
6-
branches: [main]
3+
workflow_run:
4+
workflows: ["Lint"]
5+
types:
6+
- completed
77

88
jobs:
99
primitives:
1010
name: ${{ matrix.primitive }}
1111
runs-on: ubuntu-latest
12+
if: ${{ github.event.workflow_run.conclusion == 'success' }}
1213
strategy:
1314
fail-fast: false
1415
matrix:
@@ -32,10 +33,6 @@ jobs:
3233
# Install project with dev and optional extras similar to unit tests environment
3334
uv pip install --system .[dev,anthropic,openai,search,e2b]
3435
35-
- name: Lint with ruff
36-
run: |
37-
ruff check .
38-
3936
- name: Run integration tests for ${{ matrix.primitive }} primitive
4037
run: |
4138
pytest tests/integration/primitives/test_${{ matrix.primitive }}.py

.github/workflows/transportstests.yml

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,15 @@
11
name: Transports
22
on:
3-
push:
4-
branches: [main]
5-
pull_request:
6-
branches: [main]
3+
workflow_run:
4+
workflows: ["Lint"]
5+
types:
6+
- completed
77

88
jobs:
99
transports:
1010
name: "${{ matrix.transport }}"
1111
runs-on: ubuntu-latest
12+
if: ${{ github.event.workflow_run.conclusion == 'success' }}
1213
strategy:
1314
fail-fast: false
1415
matrix:
@@ -32,10 +33,6 @@ jobs:
3233
# Install project with dev and optional extras similar to unit tests environment
3334
uv pip install --system .[dev,anthropic,openai,search,e2b]
3435
35-
- name: Lint with ruff
36-
run: |
37-
ruff check .
38-
3936
- name: Run integration tests for ${{ matrix.transport }} transport
4037
run: |
4138
pytest tests/integration/transports/${{ matrix.transport }}

.github/workflows/unittests.yml

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,15 @@
11
name: Python Tests
22

33
on:
4-
push:
5-
branches: [main]
6-
pull_request:
7-
branches: [main]
4+
workflow_run:
5+
workflows: ["Lint"]
6+
types:
7+
- completed
88

99
jobs:
1010
test:
1111
runs-on: ubuntu-latest
12+
if: ${{ github.event.workflow_run.conclusion == 'success' }}
1213
strategy:
1314
matrix:
1415
python-version: ["3.11", "3.12"]
@@ -23,9 +24,6 @@ jobs:
2324
run: |
2425
python -m pip install --upgrade pip
2526
pip install .[dev,anthropic,openai,search,e2b]
26-
- name: Lint with ruff
27-
run: |
28-
ruff check .
2927
- name: Test with pytest
3028
run: |
3129
pytest tests/unit

0 commit comments

Comments
 (0)