Skip to content

Add Redis Output Interfacer#27

Merged
mano8 merged 16 commits intomainfrom
AppsDev
Sep 3, 2024
Merged

Add Redis Output Interfacer#27
mano8 merged 16 commits intomainfrom
AppsDev

Conversation

@mano8
Copy link
Copy Markdown
Member

@mano8 mano8 commented Sep 3, 2024

Changes:

  • improve code
  • correct bugs
  • set functional Redis Output worker

Copy link
Copy Markdown
Member Author

@mano8 mano8 left a comment

Choose a reason for hiding this comment

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

ok

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 3, 2024

Codecov Report

Attention: Patch coverage is 92.18750% with 10 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
vemonitor_m8/workers/redis/redis_h_time_series.py 92.85% 6 Missing ⚠️
vemonitor_m8/workers/redis/redis_worker.py 92.30% 3 Missing ⚠️
vemonitor_m8/core/async_app_run.py 0.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
vemonitor_m8/version.py 100.00% <100.00%> (ø)
vemonitor_m8/workers/redis/redis_api.py 91.42% <ø> (ø)
vemonitor_m8/workers/redis/redis_cache.py 93.54% <100.00%> (+0.14%) ⬆️
vemonitor_m8/core/async_app_run.py 70.55% <0.00%> (ø)
vemonitor_m8/workers/redis/redis_worker.py 94.00% <92.30%> (+3.48%) ⬆️
vemonitor_m8/workers/redis/redis_h_time_series.py 93.77% <92.85%> (-0.13%) ⬇️

... and 1 file with indirect coverage changes

@mano8 mano8 merged commit 3fa5f85 into main Sep 3, 2024
Sign up for free to 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.

1 participant