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

refactor(flipt): upgrade Flipt provider to use latest 1.0.0 Flipt SDK #638

Merged
merged 5 commits into from
Jan 26, 2024

Conversation

markphelps
Copy link
Contributor

This PR

Related Issues

Closes: #636

Notes

Follow-up Tasks

How to test

@markphelps
Copy link
Contributor Author

@liran2000 could you help me debug why CI is failing please? It seems to be potentially getting a 500 from wiremock?

 Unrecognized token 'Request'

https://github.com/open-feature/java-sdk-contrib/actions/runs/7560459925/job/20586536492?pr=638#step:6:15445

Copy link
Member

@liran2000 liran2000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, sure.
Added review comments. Could not add changes to the PR, I tested the changes and it passes the tests locally at the IDE.

Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
@toddbaert
Copy link
Member

I'm seeing:

Warning:  Javadoc Warnings
Warning: [WARNING] warning: unknown enum constant Mode.PROPERTIES
Warning:  reason: class file for com.fasterxml.jackson.annotation.JsonCreator$Mode not found
Warning: [WARNING] warning: unknown enum constant Include.NON_EMPTY
Warning:  reason: class file for com.fasterxml.jackson.annotation.JsonInclude$Include not found
Warning: [WARNING] warning: unknown enum constant Mode.PROPERTIES

I'll try to look into this on monday... looks like it might be warnings coming from a dependency 🤔

Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
…java-sdk-contrib into upgrade-flipt-provider

* 'upgrade-flipt-provider' of https://github.com/flipt-io/java-sdk-contrib:
  chore(flipt): bump flipt dep
  refactor(flipt): upgrade Flipt provider to use latest 1.0.0 Flipt SDK
@toddbaert toddbaert merged commit 42738b8 into open-feature:main Jan 26, 2024
4 checks passed
@markphelps markphelps deleted the upgrade-flipt-provider branch January 26, 2024 21:17
@markphelps
Copy link
Contributor Author

Thanks all!! Much appreciated!

DBlanchard88 pushed a commit to DBlanchard88/java-sdk-contrib that referenced this pull request Apr 29, 2024
…ure#638)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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