Skip to content

Commit

Permalink
Simplify macOS testing matrix
Browse files Browse the repository at this point in the history
**What does this PR do?**:

This PR changes the `Test macOS` github action to only use the latest
macOS for testing, rather than testing with multiple versions of macOS.

**Motivation**:

We don't currently support macOS on production environments, as
documented in
<https://github.com/DataDog/dd-trace-rb/blob/master/docs/GettingStarted.md#apple-macos-support>

It's still valuable to "keep an eye on" macOS, since we do want to be
able to use it in development.

BUT our test matrix is rather complex, so I think not testing
older macOS versions * 10 Ruby versions we support is a worthy
simplification.

Additionally, once github adds support for macOS 13, we'll
automatically start testing on that one as well, which should match
what most macOS-based developers are using.

**Additional Notes**:

N/A

**How to test the change?**:

Validate that the `Test macOS` github action still runs, but only
on macos-latest (macOS 12), and does not run on macOS 11.
  • Loading branch information
ivoanjo committed Mar 1, 2023
1 parent f39b81b commit 9c475cb
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/test-macos.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@ jobs:
fail-fast: false
matrix:
os:
- macos-11
- macos-12
- macos-latest
ruby:
- '2.1'
- '2.2'
Expand Down

0 comments on commit 9c475cb

Please sign in to comment.