Skip to content

Conversation

awwaiid
Copy link
Contributor

@awwaiid awwaiid commented Dec 19, 2015

(I hate unspace)

zoffixznet added a commit that referenced this pull request Dec 19, 2015
Remove some unneeded unspace
@zoffixznet zoffixznet merged commit d5a77e6 into master Dec 19, 2015
@zoffixznet
Copy link
Contributor

Thanks a lot!

@awwaiid
Copy link
Contributor Author

awwaiid commented Dec 20, 2015

mmm... it may not have been a good idea to merge this so quickly :) . I'm checking that I didn't break anything now...

@zoffixznet
Copy link
Contributor

Sorry, I jumped the gun :)

It does seem this line now breaks the build. I think it's the same issue as when a hyper op is used with a method chain on multiple line you get errors too...

I pushed a fix.

zoffixznet added a commit that referenced this pull request Dec 20, 2015
Seems there's an issue with some of unspace-less constructs still. Should fix breakage mentioned here: #251 (comment)
@zoffixznet
Copy link
Contributor

Hm... I reverted that part, but it's still failing. no idea why

@titsuki titsuki deleted the un-unspace branch May 23, 2016 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants