fix: add guard conditions for west build tests
The RAM report, ROM report and clean steps depend on the success of the board/shield build. PR: zmkfirmware/zmk-docker#51
This commit is contained in:
		
							
								
								
									
										4
									
								
								.github/workflows/containers.yml
									
									
									
									
										vendored
									
									
								
							
							
						
						
									
										4
									
								
								.github/workflows/containers.yml
									
									
									
									
										vendored
									
									
								
							| @@ -367,11 +367,14 @@ jobs: | |||||||
|       - name: Test west zephyr-export |       - name: Test west zephyr-export | ||||||
|         run: west zephyr-export |         run: west zephyr-export | ||||||
|       - name: Test board/shield (west build) |       - name: Test board/shield (west build) | ||||||
|  |         id: board-shield | ||||||
|         if: ${{ matrix.board != null }} |         if: ${{ matrix.board != null }} | ||||||
|         run: west build -s app -b ${{ matrix.board }} -- ${{ matrix.shield != null && format('-DSHIELD={0}', matrix.shield) || null }} |         run: west build -s app -b ${{ matrix.board }} -- ${{ matrix.shield != null && format('-DSHIELD={0}', matrix.shield) || null }} | ||||||
|       - name: Test RAM report (west build) |       - name: Test RAM report (west build) | ||||||
|  |         if: ${{ steps.board-shield.outcome == 'success' }} | ||||||
|         run: west build -t ram_report |         run: west build -t ram_report | ||||||
|       - name: Test ROM report (west build) |       - name: Test ROM report (west build) | ||||||
|  |         if: ${{ steps.board-shield.outcome == 'success' }} | ||||||
|         run: west build -t rom_report |         run: west build -t rom_report | ||||||
|       - name: Test west test (single) |       - name: Test west test (single) | ||||||
|         run: west test tests/none/normal |         run: west test tests/none/normal | ||||||
| @@ -379,6 +382,7 @@ jobs: | |||||||
|         if: ${{ env.run-unit-tests == 'true' }} |         if: ${{ env.run-unit-tests == 'true' }} | ||||||
|         run: west test |         run: west test | ||||||
|       - name: Test clean (west build) |       - name: Test clean (west build) | ||||||
|  |         if: ${{ steps.board-shield.outcome == 'success' }} | ||||||
|         run: west build -t clean |         run: west build -t clean | ||||||
|       - name: Stop container |       - name: Stop container | ||||||
|         shell: bash |         shell: bash | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user