Skip to content
This repository has been archived by the owner on Apr 2, 2024. It is now read-only.

Added missing client to publish handlers in monitor #174

Merged
merged 1 commit into from
Aug 17, 2022

Conversation

icellan
Copy link
Member

@icellan icellan commented Aug 17, 2022

No description provided.

@icellan icellan requested a review from galt-tr August 17, 2022 15:00
@icellan icellan requested a review from mrz1836 as a code owner August 17, 2022 15:00
@mrz1836 mrz1836 added the hot-fix Urgent or important fix/patch label Aug 17, 2022
Copy link
Member

@mrz1836 mrz1836 left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-commenter
Copy link

Codecov Report

Merging #174 (2ed6c72) into master (7be5160) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #174   +/-   ##
=======================================
  Coverage   52.46%   52.46%           
=======================================
  Files          87       87           
  Lines        9699     9699           
=======================================
  Hits         5089     5089           
  Misses       4202     4202           
  Partials      408      408           
Flag Coverage Δ
unittests 52.46% <0.00%> (ø)

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

Impacted Files Coverage Δ
monitor_event_handler.go 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7be5160...2ed6c72. Read the comment docs.

@icellan icellan merged commit 790610f into master Aug 17, 2022
@mergify mergify bot deleted the siggi/monitor_client_missing branch August 17, 2022 16:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
hot-fix Urgent or important fix/patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants