Skip to content

feat: add java generator - #107

Merged
beeme1mr merged 8 commits into
open-feature:mainfrom
advitiya0201:feat/java-gen
May 12, 2025
Merged

feat: add java generator#107
beeme1mr merged 8 commits into
open-feature:mainfrom
advitiya0201:feat/java-gen

Conversation

@advitiya0201

Copy link
Copy Markdown
Contributor

This PR

  • Adds a Java generator to the OpenFeature CLI that generates type-safe clients compatible with the OpenFeature Java SDK
  • Includes necessary unit tests for the code changes

Related Issues

Contributes to #102

Notes

Development process was pretty smooth, kudos to a well-maintained repo structure

How to test

  • Build the CLI:
    go build
  • Generate a Java client using the sample manifest:
    ./cli generate java --manifest=sample/sample_manifest.json --package-name=com.exmaple.myRepo

@advitiya0201

Copy link
Copy Markdown
Contributor Author

Hi @beeme1mr, waiting for any comments from your side

@beeme1mr

beeme1mr commented May 4, 2025

Copy link
Copy Markdown
Member

Hey @advitiya0201 I'll review this tomorrow.

@beeme1mr beeme1mr 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.

Looks a great start. Please just add some docs to the details method and we can get this in. Thanks!

Comment thread internal/generators/java/java.tmpl Outdated
@toddbaert

Copy link
Copy Markdown
Member

I will review this tomorrow! Thanks @advitiya0201

@toddbaert
toddbaert self-requested a review May 5, 2025 18:43
Comment thread cmd/testdata/success_java.golden Outdated
Comment thread internal/config/flags.go Outdated
Signed-off-by: advitiya0201 <f20210979@pilani.bits-pilani.ac.in>
Signed-off-by: advitiya0201 <f20210979@pilani.bits-pilani.ac.in>
…va.md

Signed-off-by: advitiya0201 <f20210979@pilani.bits-pilani.ac.in>
Signed-off-by: advitiya0201 <f20210979@pilani.bits-pilani.ac.in>
Signed-off-by: advitiya0201 <f20210979@pilani.bits-pilani.ac.in>
…plementation

Signed-off-by: advitiya0201 <f20210979@pilani.bits-pilani.ac.in>
Signed-off-by: advitiya0201 <f20210979@pilani.bits-pilani.ac.in>
@beeme1mr

beeme1mr commented May 9, 2025

Copy link
Copy Markdown
Member

I'll give folks until Monday to finish their reviews.

@beeme1mr
beeme1mr added this pull request to the merge queue May 12, 2025
Merged via the queue into open-feature:main with commit 9a9f11f May 12, 2025
This was referenced Feb 6, 2026
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.

4 participants