Skip to content

Commit 3a173ab

Browse files
authored
fix: release check for integration tests (#400)
1 parent b90346c commit 3a173ab

1 file changed

Lines changed: 15 additions & 6 deletions

File tree

‎.github/workflows/release.yaml‎

Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -117,12 +117,21 @@ jobs:
117117
- name: Run integration tests
118118
run: uv run pytest tests/*/integration/ -v --tb=short
119119

120-
# ── 3. Publish ────────────────────────────────────────────────────────────
121-
publish:
122-
name: Publish
120+
# ── 3. Gate (absorbs skipped integration-tests so publish is never auto-skipped) ──
121+
gate:
122+
name: Gate
123123
needs: [parse, integration-tests]
124124
if: always() && needs.parse.result == 'success' && (needs.integration-tests.result == 'success' || needs.integration-tests.result == 'skipped')
125125
runs-on: ubuntu-latest
126+
steps:
127+
- run: echo "Gate passed"
128+
129+
# ── 4. Publish ────────────────────────────────────────────────────────────
130+
publish:
131+
name: Publish
132+
needs: [parse, gate]
133+
if: needs.gate.result == 'success'
134+
runs-on: ubuntu-latest
126135
environment: 'pypi:sap-cloud-sdk'
127136
permissions:
128137
contents: write
@@ -181,11 +190,11 @@ jobs:
181190
- name: Publish to PyPI
182191
uses: pypa/gh-action-pypi-publish@v1.14.2
183192

184-
# ── 4. Notify on failure ──────────────────────────────────────────────────
193+
# ── 5. Notify on failure ──────────────────────────────────────────────────
185194
notify-failure:
186195
name: Notify Failure
187-
needs: [parse, integration-tests, publish]
188-
if: always() && (needs.parse.result == 'failure' || needs['integration-tests'].result == 'failure' || needs.publish.result == 'failure')
196+
needs: [parse, gate, publish]
197+
if: always() && (needs.parse.result == 'failure' || needs.gate.result == 'failure' || needs.publish.result == 'failure')
189198
runs-on: ubuntu-latest
190199
permissions:
191200
contents: write

0 commit comments

Comments
 (0)