search

capaj / require-globify

transform for browserify, which allows to require files with globbing expressions
MIT License
70 stars 11 forks source link

Develop #1

Closed call-a3 closed 9 years ago

call-a3 commented 9 years ago

Switched implementation from regex matching to a more syntactical-aware one, using the falafel tool available through browserify-transform-tools Also added the requested 'hash' feature. Adjusted version number according to semver because the API was adjusted in a non-backwards-compatible way.

capaj commented 9 years ago

wow, this looks great, you should definitely be a maintainer, will test this out on our codebase and report back when I know more.

call-a3 commented 9 years ago

Important to note is that the API was changed, so requireGlob('some-pattern') will no longer work. Instead, refactor your codebase to use require('some-pattern', {glob:true}). Unless of course you prefer I make a change to make the new implementation be backwards compatible? I just figured require(.., {glob:true}) was cleaner...

capaj commented 9 years ago

works fine, merged

capaj commented 9 years ago

lets not worry about backwards compatibility- migrating to 1.0.0 is very easy(basically one fairly simple regexp search and replace) and I think this package does not have yet the userbase to justify lingering on old syntax. But hopefully with this update, we mark it 1.0.0 with confidence and release it into the wild.