Skip to content
This repository was archived by the owner on Oct 4, 2020. It is now read-only.

Conversation

@paf31
Copy link
Contributor

@paf31 paf31 commented Apr 4, 2017

I couldn't update the tests because of the test dependencies. Also, I just deleted the IsForeign instances, not sure if you wanted to do that or something else.

@paf31 paf31 requested a review from garyb April 4, 2017 02:37
@garyb
Copy link
Member

garyb commented Apr 4, 2017

Thanks! I'll probably disable the tests for a release and reinstate them when the updates are done.

Going to update the FFI stuff to use the uncurried things now too!

@garyb garyb merged commit 4f0b26a into master Apr 4, 2017
@garyb
Copy link
Member

garyb commented Apr 4, 2017

Actually I'm not updating the FFI just now, that's crazy talk. It'll take quite a while to do, and until we get fancy with the optimizer for EffFn it's not really very interesting anyway.

@garyb garyb deleted the 0.11 branch April 4, 2017 22:48
@paf31
Copy link
Contributor Author

paf31 commented Apr 5, 2017

So shall we release this as it is then?

@garyb
Copy link
Member

garyb commented Apr 5, 2017

Will do in a moment, #88 :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants