Skip to content

Conversation

@dengfeige
Copy link
Member

No description provided.

@codecov-commenter
Copy link

Codecov Report

Merging #15 (2b3792a) into main (ac77af4) will decrease coverage by 0.60%.
The diff coverage is 61.81%.

❗ Current head 2b3792a differs from pull request most recent head 4137daf. Consider uploading reports for the commit 4137daf to get more accurate results

Impacted file tree graph

@@ Coverage Diff @@## main #15 +/- ## ========================================== - Coverage 85.64% 85.03% -0.61%  ========================================== Files 34 34 Lines 1247 1270 +23 ========================================== + Hits 1068 1080 +12 - Misses 179 190 +11 
Impacted FilesCoverage Δ
featureprobe/memory_data_repository.py81.81% <0.00%> (ø)
featureprobe/pooling_synchronizer.py76.47% <ø> (ø)
featureprobe/default_event_processor.py72.09% <45.83%> (-2.31%)⬇️
featureprobe/client.py84.31% <62.50%> (-10.56%)⬇️
featureprobe/user.py91.66% <92.30%> (-1.44%)⬇️
featureprobe/__init__.py100.00% <100.00%> (ø)

Copy link
Contributor

@hezeanhezean left a comment

Choose a reason for hiding this comment

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

LGTM

@dengfeigedengfeige merged commit bfa445e into mainSep 23, 2022
Sign up for freeto 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.

4 participants

@dengfeige@codecov-commenter@hezean