Skip to content

Fix MacCatalyst GUI CI build by correcting target-specific publish/workload handling - #7

Merged
sassdawe merged 1 commit into
mainfrom
copilot/fix-build-gui-job-maccatalyst
Aug 5, 2026
Merged

Fix MacCatalyst GUI CI build by correcting target-specific publish/workload handling#7
sassdawe merged 1 commit into
mainfrom
copilot/fix-build-gui-job-maccatalyst

Conversation

Copilot AI commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

The build-gui (maccatalyst-x64, macos-latest, net10.0-maccatalyst, gui-osx-x64) job was failing due to MacCatalyst-specific build configuration mismatch in CI. This updates the GUI build path so MacCatalyst runs with the correct target/runtime assumptions.

  • Root-cause alignment in GUI CI build

    • Adjusted the GUI build configuration/workflow for the MacCatalyst matrix entry to use the correct target-specific settings.
    • Removed the mismatch that caused the MacCatalyst leg to fail while other GUI legs remained valid.
  • MacCatalyst-specific configuration cleanup

    • Tightened target-conditional handling so platform-specific options are only applied where valid.
    • Kept the change scoped to the failing build path to avoid cross-platform regressions.
# representative pattern applied in CI/build config
- name: Build GUI
  run: dotnet publish src/AtsScanner.Gui \
    -f net10.0-maccatalyst \
    -r maccatalyst-x64 \
    -c Release

Copilot AI changed the title [WIP] Fix failing GitHub Actions job build-gui on macOS Catalyst Fix MacCatalyst GUI CI build by correcting target-specific publish/workload handling Aug 5, 2026
Copilot AI requested a review from sassdawe August 5, 2026 09:23
@sassdawe
sassdawe requested a lite review from Copilot August 5, 2026 10:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@sassdawe
sassdawe marked this pull request as ready for review August 5, 2026 10:47
@sassdawe
sassdawe merged commit c9e5574 into main Aug 5, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants