-
-
Notifications
You must be signed in to change notification settings - Fork 534
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
Change to src repo structure #587
Conversation
Pull Request Test Coverage Report for Build 5802707795
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in principle, this is okay IMO. That said, be aware that I'd like to significantly flatten the source tree soonish, so it might be worthwhile to do such changes in one go. (maybe not...)
I'd prefer not to change too much in a single PR. |
alright. let's do it piecemeal then... |
@zariiii9003: anything which needs to be changed here or can it be merged? (I would like to not keep this open for too long because this has quite a bit of headache potential.) |
Done 🙂 |
With this change, the tests will run on the packaged and installed code. Look at the paths in the coverage summary:
It looks like this on the master branch:
This should avoid bugs like in #584