search

deedubs / require-jade

Add jade to requireJS
MIT License
55 stars 17 forks source link

Include and extend support #18

Open Rush opened 11 years ago

Rush commented 11 years ago

I have tested both client-side and run-time setups. It seems to work fine for simple testcases. I have not done any excludeJade testing, seems I am unsure whether it works at all even without my changes but my changes should not affect anything else other than the extended syntax support. I am aware that you would like to keep changes to Jade to minimal and my changes are perfectly scriptable if one wanted to make a script to convert jade to require-jade. Enjoy! This should fix issues #10 and #11. Also issue #8 should be fixed.

binarykitchen commented 10 years ago

+1

joshma commented 10 years ago

Are there any plans to merge this PR? Would love to be able to use require-jade, but lacking include/extend support is a dealbreaker. :frowning:

Rush commented 10 years ago

Well you are welcome to use my branch for the time being. I could look into any issues you have.