Skip to content

Conversation

@phra
Copy link
Member

@phra phra commented Jul 23, 2016

No description provided.

@phra phra force-pushed the fix/coverage branch 2 times, most recently from 7739dca to a3408e1 Compare July 27, 2016 22:28
@Foxandxss
Copy link
Member

I am not able to reproduce the error anymore. I just put a new src/app/home folder without tests and is getting covered...

@Foxandxss
Copy link
Member

I can now, but better if instead of starting at root . we put it at ./app.

@phra
Copy link
Member Author

phra commented Jul 28, 2016

ok, that's fine for me. i'll update the PR accordingly. 👍

@phra
Copy link
Member Author

phra commented Jul 31, 2016

@Foxandxss can we merge this PR now?

@Foxandxss Foxandxss merged commit 2f64fc9 into preboot:master Jul 31, 2016
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.

2 participants