Skip to content

Commit

Permalink
Refactor build.yml for distinct MATLAB/Octave artifacts
Browse files Browse the repository at this point in the history
Updated the matrix strategy outputs and steps in build.yml to
separately handle MATLAB and Octave artifacts on Windows. Changed
step IDs to `prepare_windows_matlab_artifact` and
`prepare_windows_octave_artifact` for clarity and proper artifact
management.
  • Loading branch information
ptahmose committed Oct 18, 2024
1 parent 9676534 commit 8dab688
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,8 @@ jobs:
os: [ubuntu-latest, windows-latest] # Windows and Linux builds
outputs:
linuxx64_artifact: ${{ steps.prepare_linux_artifact.outputs.linuxx64_build }}
windowsx64_artifact: ${{ steps.prepare_windows_artifact.outputs.windowsx64_build }}
windowsx64_matlab_artifact: ${{ steps.prepare_windows_matlab_artifact.outputs.windowsx64_matlab_build }}
windowsx64_octave_artifact: ${{ steps.prepare_windows_octave_artifact.outputs.windowsx64_octave_build }}
steps:
- name: Check out repository
uses: actions/checkout@v4
Expand Down Expand Up @@ -86,7 +87,7 @@ jobs:
cp build/MEXlibCZI/Release/libmexlibczi.dll %GITHUB_WORKSPACE%\OctaveMex
dir %GITHUB_WORKSPACE%\OctaveMex
- name: Prepare MATLAB-mex artifact (Windows)
id: prepare_windows_artifact
id: prepare_windows_matlab_artifact
if: runner.os == 'Windows'
shell: bash
run: |
Expand All @@ -103,7 +104,7 @@ jobs:
path: ${{ env.artifactPath }}/
name: ${{ env.artifactName }}
- name: Prepare Octave-mex artifact (Windows)
id: prepare_windows_artifact
id: prepare_windows_octave_artifact
if: runner.os == 'Windows'
shell: bash
run: |
Expand Down

0 comments on commit 8dab688

Please sign in to comment.