8000 fix(evaluation): extract environment fallback logic to helper method by erka · Pull Request #4959 · flipt-io/flipt · GitHub
[go: up one dir, main page]

Skip to content

fix(evaluation): extract environment fallback logic to helper method#4959

Merged
erka merged 1 commit intov2from
rd/v2/env-fallback-stream
Oct 29, 2025
Merged

fix(evaluation): extract environment fallback logic to helper method#4959
erka merged 1 commit intov2from
rd/v2/env-fallback-stream

Conversation

@erka
Copy link
Contributor
@erka erka commented Oct 29, 2025

Extracted duplicate environment retrieval logic with context fallback
into a reusable getCurrentEnvironment helper method. This ensures
consistent environment lookup behavior across both snapshot endpoints.

closes #4958

Extracted duplicate environment retrieval logic with context fallback
into a reusable getCurrentEnvironment helper method. This ensures
consistent environment lookup behavior across both snapshot endpoints.

Signed-off-by: Roman Dmytrenko <rdmytrenko@gmail.com>
@erka erka requested a review from a team as a code owner October 29, 2025 00:03
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Oct 29, 2025
@dosubot
Copy link
dosubot bot commented Oct 29, 2025

Documentation Updates

Checked 4 published document(s). No updates required.

How did I do? Any feedback?  Join Discord

@codecov
Copy link
codecov bot commented Oct 29, 2025

Codecov Report

❌ Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 60.29%. Comparing base (571dd72) to head (0eabe9d).
⚠️ Report is 1 commits behind head on v2.

Files with missing lines Patch % Lines
internal/server/evaluation/client/server.go 87.50% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##               v2    #4959   +/-   ##
=======================================
  Coverage   60.29%   60.29%           
=======================================
  Files         136      136           
  Lines       13372    13375    +3     
=======================================
+ Hits         8062     8065    +3     
  Misses       4629     4629           
  Partials      681      681           
Flag Coverage Δ
integrationtests 34.95% <87.50%> (+0.01%) ⬆️
unittests 51.27% <87.50%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Collaborator
@markphelps markphelps 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 i'll create a new release

@erka erka merged commit d9236db into v2 Oct 29, 2025
31 checks passed
@erka erka deleted the rd/v2/env-fallback-stream branch October 29, 2025 12:25
@erka erka added the v2 Flipt v2 label Oct 29, 2025
markphelps added a commit that referenced this pull request Oct 29, 2025
* v2:
  fix(evaluation): extract environment fallback logic to helper method (#4959)
  fix(config): use snake_case for fetch policy in json schema (#4957)
  chore: prep for v2.3.0 release (#4956)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files. v2 Flipt v2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

EvaluationSnapshotNamespaceStream 'environment: "default" not found errors' when upgrading to 2.3.0

2 participants

0