-
Notifications
You must be signed in to change notification settings - Fork 15
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
Refactored package structure: use sublibraries #115
Conversation
3e9279e
to
ae78bbd
Compare
79e55fe
to
a0faf60
Compare
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.
LGTM, needs a changelog entry
It would be nice to get |
One alternative would be to split the tests, moving the ones for IO to |
@jorisdral done;
|
`strict-mvar:test` is left in `test-strict-mvar` as it introduces a cycle between `io-sim` and `io-classes` which cabal cannot resolve.
Move `IO` tests to `io-classes` and `IOSim` tests to `io-sim`.
Put
si-timers
,strict-stm
andstrict-mvar
as public libraries ofio-classes
.strict-mvar:test
is left intest-strict-mvar
as it introducesa cycle between
io-sim
andio-classes
which cabal cannot resolve.