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

Fix test on Windows - issue with new line character #63

Merged
merged 1 commit into from
Sep 10, 2016

Conversation

stonio
Copy link
Contributor

@stonio stonio commented Sep 6, 2016

Fix test on Windows platform by using NEWLINE instead of \n.

Fix test on Windows platform by using NEWLINE instead of '\n'.
@codecov-io
Copy link

Current coverage is 80% (diff: 100%)

Merging #63 into master will increase coverage by <1%

@@             master        #63   diff @@
==========================================
  Files           207        207          
  Lines          8079       8079          
  Methods           0          0          
  Messages          0          0          
  Branches       1377       1377          
==========================================
+ Hits           6487       6488     +1   
  Misses         1382       1382          
+ Partials        210        209     -1   

Powered by Codecov. Last update 4639361...4e959f0

@mangstadt mangstadt merged commit d6a31c5 into mangstadt:master Sep 10, 2016
@mangstadt
Copy link
Owner

Thank you for catching this.

@stonio stonio deleted the patch-1 branch September 12, 2016 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants