-
Notifications
You must be signed in to change notification settings - Fork 17
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
fix(kotlin): macos compatible helpers and build on CI #3297
Conversation
7f4f789
to
acfc9fb
Compare
✔️ Code generated!
|
if: | | ||
always() && | ||
(needs.swift_cts_macos.result == 'success' || needs.swift_cts_macos.result == 'skipped') && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure about this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup it should already be validate by the next clause I think
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's go with that
algolia/api-clients-automation#3297 Co-authored-by: Clément Vannicatte <[email protected]>
🧭 What and Why
🎟 JIRA Ticket: https://algolia.atlassian.net/browse/DI-2535
Changes included:
build fail on macos https://github.com/algolia/algoliasearch-client-kotlin/actions/runs/9699224976/job/26767673731 but looks good on ubuntu
this is because we use non native features in some helpers, now our CI builds both version so we don't make the same mistake again