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

[9538] Fix eventAsText epytext #1069

Merged
merged 3 commits into from Sep 25, 2018
Merged

[9538] Fix eventAsText epytext #1069

merged 3 commits into from Sep 25, 2018

Conversation

twm
Copy link
Contributor

@twm twm commented Sep 23, 2018

epydoc didn't like the newlines in the middle of the C{u"{timeStamp} [{system}] {event}\n{traceback}\n"} literal. Making the docstring a raw string changes them to the backslash escapes that were intended anyway.

Contributor Checklist:

@codecov
Copy link

codecov bot commented Sep 24, 2018

Codecov Report

Merging #1069 into trunk will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            trunk    #1069      +/-   ##
==========================================
+ Coverage   91.92%   91.92%   +<.01%     
==========================================
  Files         844      844              
  Lines      150865   150865              
  Branches    13157    13157              
==========================================
+ Hits       138680   138684       +4     
+ Misses      10099    10096       -3     
+ Partials     2086     2085       -1

Copy link
Member

@altendky altendky left a comment

Choose a reason for hiding this comment

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

Please merge.

@twm
Copy link
Contributor Author

twm commented Sep 25, 2018

Thank you for the review, @altendky!

@twm twm merged commit 89bd4f3 into trunk Sep 25, 2018
@twm twm deleted the 9538-eventastext-epytext branch September 25, 2018 04:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants