Skip to content

Conversation

@BYK
Copy link
Member

@BYK BYK commented Nov 19, 2024

Follow up to #3427 (comment)

@BYK BYK requested a review from mwarkentin November 19, 2024 14:38
@BYK BYK enabled auto-merge (squash) November 19, 2024 14:39
@codecov
Copy link

codecov bot commented Nov 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.01%. Comparing base (c3814f0) to head (01867ec).
Report is 1 commits behind head on master.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3432   +/-   ##
=======================================
  Coverage   99.01%   99.01%           
=======================================
  Files           3        3           
  Lines         203      203           
=======================================
  Hits          201      201           
  Misses          2        2           

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

@BYK BYK requested a review from mwarkentin November 19, 2024 16:10
Copy link
Member

@mwarkentin mwarkentin left a comment

Choose a reason for hiding this comment

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

LGTM!

@BYK BYK merged commit 0b0d0c8 into master Nov 19, 2024
14 checks passed
@BYK BYK deleted the byk/fix/safer-redis-eviction branch November 19, 2024 16:34
@github-actions github-actions bot locked and limited conversation to collaborators Dec 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants