-
Notifications
You must be signed in to change notification settings - Fork 38
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
chore(deps): update open-feature/flagd #600
Conversation
c205a4f
to
25b6223
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #600 +/- ##
=======================================
Coverage 88.22% 88.22%
=======================================
Files 10 10
Lines 960 960
=======================================
Hits 847 847
Misses 88 88
Partials 25 25
Flags with carried forward coverage won't be shown. Click here to find out more. |
69773c7
to
548ff81
Compare
548ff81
to
344fa9e
Compare
Signed-off-by: Todd Baert <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This updates OFO to use the latest flagd and flagd-proxy versions, which use the new protos.
Since OFO doesn't yet support in-process providers, I think this should "just work", since flagd uses the new sync proto (as a client) and can talk to flagd-proxy which uses the new proto (as a server).
flagd itself still supports both the old and new evaluation protos, so for end-users nothing breaks with this (their workloads can still talk to flagd just fine).
Signed-off-by: Todd Baert <[email protected]>
@toddbaert renovate failed as current regex doesn't cover above files [1]. I think this is due to recent refactorings which didn't update the pattern. I think we should update this to include any [1] - https://github.com/open-feature/open-feature-operator/blob/main/renovate.json#L41 |
Ah, I see. Thanks. I'm fine with that. Do you think we should include it here or in a new PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I will open a quick PR for this :) Update - please see here #607 |
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. ⚠ Warning: custom changes will be lost. |
This PR contains the following updates:
v0.7.2
->v0.9.0
v0.3.2
->v0.5.0
Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Release Notes
open-feature/flagd (ghcr.io/open-feature/flagd-proxy)
v0.4.2
Compare Source
🧹 Chore
e0b9d18
(#474) (5b85b2a)🐛 Bug Fixes
✨ New Features
v0.4.1
Compare Source
🔄 Refactoring
🧹 Chore
bd2d118
(#471) (ee90f48)🐛 Bug Fixes
v0.4.0
Compare Source
⚠ BREAKING CHANGES
🧹 Chore
fc89b04
(#417) (04014e7)b6bca70
(#441) (ce0ebe1)ec9cdf0
(#437) (2650670)3343011
(#438) (e7ebf32)32dc499
(#439) (f91d91b)d3c71f9
(#406) (6e1ffb2)🐛 Bug Fixes
✨ New Features
📚 Documentation
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR has been generated by Mend Renovate. View repository job log here.