Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: fix icu4c on macos and CIFuzz test #1294

Closed
wants to merge 1 commit into from

Conversation

charlievieth
Copy link
Contributor

Fix an issue with icu4c on macos that was caused by a recent change to the formula's name and alias: Homebrew/homebrew-core#169239 (comment) and update the CIFuzz test to use V4 of the upload-artifact GH action.

@charlievieth
Copy link
Contributor Author

@mattn or @rittneje could either of you review this PR since it fixes the CI tests for outstanding PRs?

@rittneje
Copy link
Collaborator

rittneje commented Dec 9, 2024

@charlievieth I'm not familiar with the GitHub workflow. How exactly is icu4c getting installed today? Is it just there by default somehow?

@charlievieth
Copy link
Contributor Author

@rittneje This appears to have been a transient error due brew's transition to icu4@76 so I'll close this PR now that other PR's are pasing without this fix.

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.

2 participants