search

deedubs / require-jade

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

Upgraded jade dependency to 0.30.0 #22

Closed crissdev closed 10 years ago

crissdev commented 10 years ago

The jade dependency was upgraded to 0.30.0 I couldn't upgrade it to the latest version because there is a dependency hard to refactor - such as uglify-js

I didn't update the version in the package.json because it was already set to 0.0.2, though the file jade.js was specifying version 0.0.1

deedubs commented 10 years ago

LGTM @vincentmac?

vincentmac commented 10 years ago

@CrissDev Thanks. I'll take a look at this in the morning. If I don't get back to you by mid-afternoon (PST), please mention me again in this PR so I get a notification (and a reminder ;) ).

deedubs commented 10 years ago

Thanks @CrissDev & @vincentmac

crissdev commented 10 years ago

@vincentmac @deedubs Thanks for merging this. I'll try to push more updates in the future.