abnobdoss opened a new issue, #1261:
URL: https://github.com/apache/iceberg-go/issues/1261
## Summary
`go-ci.yml` advertises a multi-OS matrix, but every job runs on Linux, so
Windows and macOS are never actually tested.
## Current behavior
- `runs-on:` is hardcoded to `ubuntu-latest` while `name:` uses `${{
matrix.os }}`, so the `os` axis has no effect on where jobs run.
- The matrix just runs duplicate Linux jobs; macOS and Windows builds never
happen.
## Expected behavior
- Each `os` value runs on its own runner, so macOS (and later Windows) is
actually tested.
## Proposed fix
- Set `runs-on: ${{ matrix.os }}`. Enable macOS now. Windows needs path
fixes first (separate issue), so leave it commented out for the moment.
- Add `.gitattributes` with `*.go text eol=lf` so gofmt doesn't fail on CRLF
checkouts once non-Linux runners are in play. It's a no-op on Linux/macOS.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
