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

Replace logrus with slog in handlers packages #659

Merged
merged 1 commit into from
Oct 12, 2023
Merged

Replace logrus with slog in handlers packages #659

merged 1 commit into from
Oct 12, 2023

Conversation

norkans7
Copy link
Contributor

contains #658

@codecov
Copy link

codecov bot commented Oct 12, 2023

Codecov Report

Merging #659 (b4ea6ab) into main (1b14a95) will not change coverage.
The diff coverage is 16.66%.

@@           Coverage Diff           @@
##             main     #659   +/-   ##
=======================================
  Coverage   74.04%   74.04%           
=======================================
  Files         100      100           
  Lines       13177    13177           
=======================================
  Hits         9757     9757           
  Misses       2732     2732           
  Partials      688      688           
Files Coverage Δ
handlers/twiml/handlers.go 80.85% <100.00%> (ø)
handlers/facebook_legacy/handler.go 84.37% <0.00%> (ø)
handlers/hormuud/handler.go 80.73% <0.00%> (ø)
handlers/messagebird/handler.go 73.40% <0.00%> (ø)
handlers/whatsapp_legacy/handler.go 79.26% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@rowanseymour rowanseymour merged commit 6368e43 into main Oct 12, 2023
6 of 7 checks passed
@rowanseymour rowanseymour deleted the slog-2 branch October 12, 2023 15:57
@github-actions github-actions bot locked and limited conversation to collaborators Oct 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants