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

[WIP] Port to reporter #230

Merged
merged 12 commits into from
Aug 18, 2018
Merged

[WIP] Port to reporter #230

merged 12 commits into from
Aug 18, 2018

Conversation

ljvmiranda921
Copy link
Owner

Port module to use Reporter

@ljvmiranda921 ljvmiranda921 added enhancement Feature requests v.1.1.0 In pipeline for next version labels Aug 17, 2018
@ljvmiranda921 ljvmiranda921 self-assigned this Aug 17, 2018
@ljvmiranda921
Copy link
Owner Author

Please use logger.exception for all raise calls so that the whole stack trace is included in the log. self.rep.exception()

@ljvmiranda921 ljvmiranda921 merged commit 57b8dcd into add-reporter-module Aug 18, 2018
@ljvmiranda921 ljvmiranda921 deleted the port-to-reporter branch August 18, 2018 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ljvmiranda921 ljvmiranda921

Labels
enhancement Feature requests v.1.1.0 In pipeline for next version
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ljvmiranda921