Skip to content

Conversation

@fakraemer
Copy link
Contributor

IOW.bind was eagerly running its IO, changed this syntax interface to just reuse IOFunctions.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 34.297% when pulling 77258ff on fakraemer:master into b707324 on functionaljava:master.

@jbgi
Copy link
Member

jbgi commented May 23, 2016

👍 well spotted!
You may want to associate your Atlassian email with your github account so that the commit properly link to it.

@fakraemer
Copy link
Contributor Author

Thanks, I forgot that I committed with my work config. Associated the account now. Do I need to worry about the coverall build?

@jbgi jbgi merged commit cbeec83 into functionaljava:master May 23, 2016
@jbgi
Copy link
Member

jbgi commented May 23, 2016

no worry. I don't understand why coverage would decrease.

@fakraemer
Copy link
Contributor Author

@jbgi would you mind cutting a release by any chance? I'd like to make some of our code a bit more readable by using the syntax interface.

@jbgi
Copy link
Member

jbgi commented May 26, 2016

I also think it is time, almost: I'd like we have a fix for #257 and #251 merged before.
@mperry what do you think?

@mperry
Copy link
Contributor

mperry commented Jun 6, 2016

The last release was mid Feb, so it has been 4 months. I usually give about 1.5 weeks notice, so perhaps we should target Sat/Sun Jun 18/19.

@mperry mperry added the bug label Jul 1, 2016
@mperry mperry added this to the v4.6 milestone Jul 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants