-
Notifications
You must be signed in to change notification settings - Fork 11
Conversation
I just cannot get this project to build after applying the patch. I take we have to upgrade I took a quick stab at it but it winged that prelude was defined multiple times now? |
Did you clear out |
It says |
I guess this would also be pending a purescript 0.7 release to npm. |
I filed an issue for this: purescript-deprecated/gulp-purescript#50 |
I don't use |
👍 for the changes. |
Can I merge this then? Seems the |
Personally I would wait until it's in a compilable state before merging into master. I am going to look into this. Just not too sure when, my second daughter just got born hours ago :) either way my vote is to keep this on a branch until ready. @garyb what do u reckon? |
Congratulations :D |
Congrats! 🎉 |
I've made a couple of changes, including an update to the gulpfile, the way the docs are generated and so on. I made sure to create atomic commits so it's easier to get an overview. We can squash before merge if so desired. It runs fine after |
Looks fine, but I don't use |
Can we get this merged then? |
I don't see why not, I will take care of it. |
/cc @garyb @felixSchl
This is a minimal set of changes.