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.

Sign up for GitHub

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

Jump to bottom

chore(flagd): Updates flagd core to v.0.11.1, taking care of breaking changes #627

Merged
merged 4 commits into from
Feb 18, 2025
Merged

chore(flagd): Updates flagd core to v.0.11.1, taking care of breaking changes #627

merged 4 commits into from
Feb 18, 2025

Conversation

cupofcat
Copy link
Contributor

This PR

Related Issues

@cupofcat cupofcat requested a review from a team as a code owner February 17, 2025 12:46
@cupofcat cupofcat changed the title chore: Updates flagd core to v.0.11.1, taking care of breaking changes chore(flagd): Updates flagd core to v.0.11.1, taking care of breaking changes Feb 17, 2025
aepfli
aepfli approved these changes Feb 17, 2025
View reviewed changes
Copy link
Member

@aepfli aepfli left a comment

Choose a reason for hiding this comment

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

Thank you! from the perspective of a golang-noob a I am, this looks reasonable and perfectly fine. @open-feature/sdk-golang-maintainers your insights here would be highly valuable

@cupofcat cupofcat mentioned this pull request Feb 17, 2025
Closed
1 task
@tangenti tangenti mentioned this pull request Feb 18, 2025
Closed
@cupofcat
Copy link
Contributor Author

I would suggest we merge this PR and then fix the linter errors separately (as per #628)

@beeme1mr beeme1mr merged commit e3d5fc7 into open-feature:main Feb 18, 2025
5 checks passed
@github-actions github-actions bot mentioned this pull request Feb 18, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@beeme1mr beeme1mr beeme1mr approved these changes

@aepfli aepfli aepfli approved these changes

@bacherfl bacherfl Awaiting requested review from bacherfl

@Kavindu-Dodan Kavindu-Dodan Awaiting requested review from Kavindu-Dodan

@toddbaert toddbaert Awaiting requested review from toddbaert

Assignees

@bacherfl bacherfl

@Kavindu-Dodan Kavindu-Dodan

@toddbaert toddbaert

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@cupofcat @beeme1mr @aepfli @bacherfl @Kavindu-Dodan @toddbaert