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.

Sign up for GitHub

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

Jump to bottom

Update logging #95

Merged
merged 5 commits into from
Jul 22, 2019
Merged

Update logging #95

merged 5 commits into from
Jul 22, 2019

Conversation

PFigs
Copy link
Contributor

@PFigs PFigs commented Jul 22, 2019

No description provided.

@PFigs PFigs added the interface/transport Issues related to the gateway transport interface label Jul 22, 2019
@PFigs PFigs requested a review from GwendalRaoul July 22, 2019 10:14
Pedro Silva added 4 commits July 22, 2019 15:02
Logger helper class
1009fc0 
This commit cleans up the logger setup by adding a class to
handle the object creation.
Print exception error
3fed983
Disallow log propagation to avoid duplicate output
e25d873
Address codacy comments
18b9b47
@PFigs PFigs merged commit 4138272 into master Jul 22, 2019
@PFigs PFigs deleted the update-logging branch July 22, 2019 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hound Hound Hound left review comments

@GwendalRaoul GwendalRaoul GwendalRaoul approved these changes

Assignees

@PFigs PFigs

@GwendalRaoul GwendalRaoul

Labels
interface/transport Issues related to the gateway transport interface
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@PFigs @GwendalRaoul