Skip to content

[ga] Add test for macos-14 and macos-15 (Apple Silicon)#648

Closed
iory wants to merge 5 commits into
euslisp:masterfrom
iory:apple-silicon-test
Closed

[ga] Add test for macos-14 and macos-15 (Apple Silicon)#648
iory wants to merge 5 commits into
euslisp:masterfrom
iory:apple-silicon-test

Conversation

@iory

@iory iory commented Jul 1, 2025

Copy link
Copy Markdown
Contributor

Related to euslisp/EusLisp#529
This PR enables tests for macos-14 and macos-15 (apple-silicon).

This test is expected to fail on macos-14 and macos-15.

The following PR (#647) applies a patch to make the test pass on macos-14 and macos-15.

@iory iory requested a review from Copilot July 1, 2025 12:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR extends the macOS CI matrix to include Apple Silicon on macOS 14 and 15 and updates the cache action to v4.

  • Add a matrix strategy for macOS jobs targeting Intel (macos-13) and ARM64 (macos-14, macos-15)
  • Upgrade actions/cache from v2 to v4
Comments suppressed due to low confidence (2)

.github/workflows/config.yml:207

  • The ::set-output command is deprecated; migrate to the new output syntax by appending to the $GITHUB_OUTPUT file, e.g., echo "dir=$(brew --cache)/downloads" >> $GITHUB_OUTPUT.
        run: echo "::set-output name=dir::$(brew --cache)/downloads"

.github/workflows/config.yml:209

  • [nitpick] Consider adding a restore-keys entry to this cache step to provide fallback keys and improve cache hit rates when the primary key misses.
        uses: actions/cache@v4

@k-okada k-okada left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@iory

iory commented Jul 2, 2025

Copy link
Copy Markdown
Contributor Author

Thank you for your review.
I merged the commit and all tests without macos-x passed.

@iory

iory commented Jul 2, 2025

Copy link
Copy Markdown
Contributor Author

Closed by #647

@iory iory closed this Jul 2, 2025
@iory iory deleted the apple-silicon-test branch July 2, 2025 07:39
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