Skip to content

Commit 0253a40

Browse files
MishaKavclaude
andcommitted
Simplify workflows: workflow_dispatch only, remove fallbacks
- Remove pull_request trigger (inputs not available) - Use inputs.branch directly (required field, always has value) - This should actually work now 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent df135fa commit 0253a40

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

.github/workflows/test-pytest-coverage-comment.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ on:
88
required: true
99
default: 'main'
1010
type: string
11-
pull_request:
1211

1312
permissions:
1413
contents: write
@@ -162,12 +161,12 @@ jobs:
162161
- name: Display test configuration
163162
run: |
164163
echo "Testing: ${{ matrix.test-name }}"
165-
echo "Branch: ${{ inputs.branch || 'main' }}"
164+
echo "Branch: ${{ inputs.branch }}"
166165
echo "Coverage type: ${{ matrix.coverage-type }}"
167166
168167
- name: Run pytest-coverage-comment
169168
id: coverageComment
170-
uses: MishaKav/pytest-coverage-comment@${{ inputs.branch || 'main' }}
169+
uses: MishaKav/pytest-coverage-comment@${{ inputs.branch }}
171170
with:
172171
pytest-xml-coverage-path: ${{ matrix.pytest-xml-coverage-path }}
173172
pytest-coverage-path: ${{ matrix.pytest-coverage-path }}

.github/workflows/test-pytest-simple.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ on:
77
description: 'Branch from pytest-coverage-comment'
88
required: true
99
default: 'main'
10-
pull_request:
1110

1211
permissions:
1312
contents: write
@@ -25,11 +24,11 @@ jobs:
2524

2625
- name: Show branch
2726
run: |
28-
echo "Testing branch: ${{ inputs.branch || 'main' }}"
27+
echo "Testing branch: ${{ inputs.branch }}"
2928
3029
- name: Run action
3130
id: coverage
32-
uses: MishaKav/pytest-coverage-comment@${{ inputs.branch || 'main' }}
31+
uses: MishaKav/pytest-coverage-comment@${{ inputs.branch }}
3332
with:
3433
pytest-xml-coverage-path: ./data/coverage_1.xml
3534
junitxml-path: ./data/pytest_1.xml

0 commit comments

Comments
 (0)