Thanks to visit codestin.com
Credit goes to github.com

Skip to content

chore: update component owners and flagd readme #1372

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

liran2000
Copy link
Member

@liran2000 liran2000 commented May 15, 2025

  • add flagd-http-connector component owner
  • add note on the flagd Java Provider readme per comment by @beeme1mr .
  • remove very old slf4j-test dependency from parent pom.
  • add com.google.guava dependency to go-feature-flag as it was using it, and CI build failed without it.

@toddbaert flagd-http-connector release-please PR was not created, can you possibly re-run release please or something to check it ?

Signed-off-by: liran2000 <[email protected]>
Signed-off-by: liran2000 <[email protected]>
@github-actions github-actions bot requested a review from thomaspoignant May 15, 2025 10:48
@liran2000 liran2000 marked this pull request as ready for review May 15, 2025 10:56
@liran2000 liran2000 requested a review from a team as a code owner May 15, 2025 10:56
@beeme1mr beeme1mr changed the title feat: updates chore: update component owners and flagd readme May 15, 2025
@beeme1mr
Copy link
Member

Hey @liran2000, it looks like we missed added the HTTP connector to the Release Please config in the PR. Could you please open a PR to add it here and here? Sorry, I should have caught this in your original PR.

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.

8 participants