Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(notifier) fix dropped notifications mentric #955

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

kissken
Copy link
Member

@kissken kissken commented Nov 7, 2023

No description provided.

@kissken kissken requested a review from a team as a code owner November 7, 2023 06:12
@codecov-commenter
Copy link

codecov-commenter commented Nov 7, 2023

Codecov Report

Merging #955 (a9f1857) into master (d0cc902) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

❗ Current head a9f1857 differs from pull request most recent head 69e49fc. Consider uploading reports for the commit 69e49fc to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##           master     #955      +/-   ##
==========================================
- Coverage   69.24%   69.24%   -0.01%     
==========================================
  Files         196      196              
  Lines       11047    11047              
==========================================
- Hits         7650     7649       -1     
- Misses       2945     2946       +1     
  Partials      452      452              
Files Coverage Δ
notifier/notifier.go 70.99% <0.00%> (-0.77%) ⬇️

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@kissken kissken merged commit 2e1545d into master Nov 8, 2023
6 checks passed
@kissken kissken deleted the fix/metric_dropped_notification branch November 8, 2023 13:32
Copy link

github-actions bot commented Nov 8, 2023

Build and push Docker images with tag: 2023-11-08.2e1545d

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.

4 participants