Skip to content

Commit 049eccf

Browse files
authored
Update ci.yml
1 parent df0180e commit 049eccf

1 file changed

Lines changed: 8 additions & 6 deletions

File tree

.github/workflows/ci.yml

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@ jobs:
3636
name: Build and test
3737
needs: semver
3838
runs-on: ${{ matrix.os }}
39+
outputs:
40+
build-succeeded: ${{ steps.upload-bundle.conclusion == 'success' }}
3941

4042
defaults:
4143
run:
@@ -101,11 +103,12 @@ jobs:
101103
-p:BaseOutputPath="${{ env.OUTPUT_PATH }}"
102104
103105
- name: Upload release bundle
106+
id: upload-bundle
104107
uses: actions/upload-artifact@v4
105108
if: ${{ matrix.publish && steps.build.conclusion == 'success' }}
106109
with:
107110
name: bundle-${{ matrix.configuration }}
108-
path: ${{ env.PUBLISH_PATH }}/${{ matrix.configuration }}
111+
path: ${{ env.OUTPUT_PATH }}/${{ matrix.configuration }}
109112

110113
- name: Test
111114
if: ${{ !matrix.coverage }}
@@ -140,8 +143,8 @@ jobs:
140143
reporter: dotnet-trx
141144

142145
report:
143-
needs: build
144-
if: always()
146+
needs: [build]
147+
if: ${{ needs.build.build-succeeded }}
145148
runs-on: ubuntu-latest
146149

147150
steps:
@@ -184,7 +187,7 @@ jobs:
184187

185188
repack-per-framework:
186189
name: Repackage per framework
187-
if: always()
190+
if: ${{ needs.build.build-succeeded }}
188191
needs: [build, semver]
189192
runs-on: ubuntu-latest
190193

@@ -219,8 +222,7 @@ jobs:
219222

220223
repack-with-plugins:
221224
name: Repackage with plugins
222-
if: always()
223-
needs: [build, semver, repack-per-framework]
225+
needs: [semver, repack-per-framework]
224226
runs-on: ubuntu-latest
225227

226228
strategy:

0 commit comments

Comments
 (0)