Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Added some missing logs #5001

Merged
merged 2 commits into from
Dec 17, 2019
Merged

Added some missing logs #5001

merged 2 commits into from
Dec 17, 2019

Conversation

maaktweluit
Copy link
Contributor

  • task-api task completed
  • task_id for offers received
  • debug logs for failing to read message queue

@maaktweluit maaktweluit self-assigned this Dec 17, 2019
kmazurek
kmazurek approved these changes Dec 17, 2019
View reviewed changes
Copy link
Contributor

@kmazurek kmazurek left a comment

Choose a reason for hiding this comment

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

Looks good, one minor suggestion.

@codecov
Copy link

codecov bot commented Dec 17, 2019 •
edited
Loading

Codecov Report

Merging #5001 into b0.22 will increase coverage by 2.25%.
The diff coverage is 80%.

@@            Coverage Diff            @@
##            b0.22   #5001      +/-   ##
=========================================
+ Coverage   87.85%   90.1%   +2.25%     
=========================================
  Files         233     233              
  Lines       21886   21890       +4     
=========================================
+ Hits        19227   19724     +497     
+ Misses       2659    2166     -493

@maaktweluit maaktweluit merged commit a133cb3 into b0.22 Dec 17, 2019
@maaktweluit maaktweluit deleted the mwu/misc-logs branch December 17, 2019 17:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kmazurek kmazurek kmazurek approved these changes

@jiivan jiivan Awaiting requested review from jiivan

@mfranciszkiewicz mfranciszkiewicz Awaiting requested review from mfranciszkiewicz

Assignees

@maaktweluit maaktweluit

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@maaktweluit @kmazurek