Skip to content

fix: Update snapshot action to run on MacOS#227

Merged
nicklasl merged 5 commits into
open-feature:mainfrom
PenguinDan:Update-Snapshot-Action
Apr 5, 2026
Merged

fix: Update snapshot action to run on MacOS#227
nicklasl merged 5 commits into
open-feature:mainfrom
PenguinDan:Update-Snapshot-Action

Conversation

@PenguinDan
Copy link
Copy Markdown
Contributor

@PenguinDan PenguinDan commented Apr 5, 2026

We need to publish on MacOS to correctly provide ios artifacts

Other updates:

  • removed legacy ways to setup gradle. The implementation aligns with my updates on release_please and the other actions

How to test

Created a test action that builds and publishes successfully to maven local and checks to see all of the artifacts have been built correctly

https://github.com/open-feature/kotlin-sdk/actions/runs/23991599947/job/69972221710

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@PenguinDan PenguinDan changed the title Update snapshot action to run on MacOS fix: Update snapshot action to run on MacOS Apr 5, 2026
@PenguinDan PenguinDan force-pushed the Update-Snapshot-Action branch from 18e80f8 to 97b7076 Compare April 5, 2026 01:24
Signed-off-by: PenguinDan <daniel.kim@includedhealth.com>
Signed-off-by: PenguinDan <daniel.kim@includedhealth.com>
Signed-off-by: PenguinDan <daniel.kim@includedhealth.com>
Made-with: Cursor
Signed-off-by: PenguinDan <daniel.kim@includedhealth.com>
@PenguinDan PenguinDan force-pushed the Update-Snapshot-Action branch from 2d107c7 to cf6fcfc Compare April 5, 2026 01:26
Signed-off-by: PenguinDan <daniel.kim@includedhealth.com>
@PenguinDan PenguinDan marked this pull request as ready for review April 5, 2026 01:32
@nicklasl nicklasl merged commit b8cb0f2 into open-feature:main Apr 5, 2026
9 checks passed
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