Fixed #16 and #20 - Githubissues

adinapoli / threads-supervisor

Simple, IO-based Haskell library for Erlang-inspired thread supervisors

MIT License

29 stars 4 forks source link

Closed adinapoli closed 8 years ago

adinapoli commented 8 years ago

cc @srijs for visibility.

See #20 for the rationale.

The deletion of a SupervisorSpec brought tons of advantages:

API simplification
87% test coverage :tada:
Renamed monitor to monitorWith and actually implemented it to supervise supervisors

Overall, seems like a great success!

Steps to migrate:

Remove any call to newSupervisorSpec
Pass to newSupervisor the RestartStrategy and, according to the flavour (bounded vs unbounded) the size as an Int
Profit!

adinapoli commented 8 years ago

Argh, will look at the Travis failure tomorrow morning.

adinapoli commented 8 years ago

Build cleared up, go figure.