Skip to content

feat(sdk): secretspec-jvm - #318

Open
moreau-nicolas wants to merge 4 commits into
cachix:mainfrom
moreau-nicolas:secretspec-jvm
Open

feat(sdk): secretspec-jvm#318
moreau-nicolas wants to merge 4 commits into
cachix:mainfrom
moreau-nicolas:secretspec-jvm

Conversation

@moreau-nicolas

Copy link
Copy Markdown

Add CI workflow.
Add doc.
Add examples.
Use gradle.
Handle MUSL libraries.
Update .gitignore.
Update scripts.
Update conformance.

Add CI workflow.
Add doc.
Add examples.
Use gradle.
Handle MUSL libraries.
Update .gitignore.
Update scripts.
Update conformance.
@moreau-nicolas

Copy link
Copy Markdown
Author

Fixes #310.

GitHub Actions workflow still needs work.

Use gradle wrapper.
Use gradle modules.
Add a consumer test.
Synchronize SDK version to a SNAPSHOT version.
Use a naive publication mechanism in GitHub Actions.
@moreau-nicolas
moreau-nicolas marked this pull request as ready for review August 14, 2026 15:02
Rename SNAPSHOT artifacts during publication.
@domenkozar

Copy link
Copy Markdown
Member

Sorry it took so long, CI is failing

@domenkozar domenkozar added the sdk label Aug 15, 2026
@gpampara

gpampara commented Aug 15, 2026

Copy link
Copy Markdown

I wonder if we need to do that xml prep step for the publishing vs just deferring to gradle to prepare the artifacts? It would be far less fragile and simpler to maintain going forward.

Fix GitHub Actions workflow errors & mistakes.
Temurin JDK 11 is not available on win32-aarch64: use Microsoft JDK.
@moreau-nicolas

Copy link
Copy Markdown
Author

@gpampara The gradle plugins I found and tried expect to build locally before publishing, which would fail to include all the platform specific libraries. The XML format is pretty stable and should not pose any maintenance headache in the future.

@domenkozar Sorry for the failing build, it is my first GitHub Action workflow ever. There will probably be other mistakes. 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants