Require forwardable - Githubissues

codegram / futuroscope

Yet another Futures implementation in Ruby

MIT License

209 stars 13 forks source link

Require forwardable #4

Closed sevos closed 11 years ago

sevos commented 11 years ago

Missing requirement was failing on Rubinius

coveralls commented 11 years ago

Coverage remained the same when pulling 743c7dc746c2a30f99ca72c541085c54a675e178 on sevos:patch-1 into e6c64d3cbb5a57e73bcfc75cafff7ac804efed02 on codegram:master.

josepjaume commented 11 years ago

Thanks for the heads up! But since we're only using the Forwardable module in future.rb, could you add it here?

https://github.com/codegram/futuroscope/blob/master/lib/futuroscope/future.rb#L2

(I'm all in for explicit dependencies)

sevos commented 11 years ago

I just did it :)

josepjaume commented 11 years ago

Yay! Sorry, I forgot to merge! Thanks :+1: