Skip to content

Add macOS coverage to Go CI#63

Draft
Copilot wants to merge 5 commits into
mainfrom
copilot/add-macos-to-build-matrix
Draft

Add macOS coverage to Go CI#63
Copilot wants to merge 5 commits into
mainfrom
copilot/add-macos-to-build-matrix

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 19, 2026

Description

CI only exercised the Go module on Linux, so darwin-only build tag regressions could merge unnoticed. This adds PR-time macOS coverage for the Go module.

  • Go CI matrix
    • Adds go-build-test with ubuntu-24.04 and macos-latest.
    • Runs both compile and test coverage:
os: [ubuntu-24.04, macos-latest]
run: go test ./...
run: go build ./...
  • Regression coverage
    • Adds a focused workflow test that asserts the macOS matrix and Go commands stay in CI.

Type of change

  • Bug fix
  • New feature
  • Documentation
  • Maintenance / refactor

Testing

  • Tested locally
  • All existing tests pass
  • Added tests for new functionality (if applicable)

Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Copilot AI changed the title [WIP] Add macOS to the Go build/test matrix Add macOS coverage to Go CI May 19, 2026
Copilot AI requested a review from mrjf May 19, 2026 15:15
Copilot AI linked an issue May 19, 2026 that may be closed by this pull request
Copilot finished work on behalf of mrjf May 19, 2026 15:37
@mrjf
Copy link
Copy Markdown
Contributor

mrjf commented May 19, 2026

@copilot fix ci

Copilot AI and others added 2 commits May 19, 2026 17:01
Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 19, 2026

@copilot fix ci

Fixed in 256297c. The macOS Go CI failure was the reflink package excluding darwin from its fallback implementation; darwin now gets the generic fallback and the package cross-compiles for macOS.

Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Copilot finished work on behalf of mrjf May 19, 2026 17:05
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.

CI: add macOS to the Go build/test matrix

2 participants