Remove non npm dependency

royriojas / esformatter-jsx

esformatter plugin: format jsx files (or js files with Facebook React JSX Syntax)

MIT License

142 stars 25 forks source link

Remove non npm dependency #40

Closed feross closed 8 years ago

feross commented 8 years ago

This package is depending on the tarball hosted here:

https://github.com/royriojas/fresh-falafel/tarball/fix-parse-issue

This is currently breaking offline installs using the local npm cache. Is there anyway not to depend on this?

(re-posted from https://github.com/feross/standard/issues/326)

feross commented 8 years ago

cc @mafintosh

max-mapper commented 8 years ago

@substack if you merge https://github.com/substack/node-falafel/pull/51 it might help

royriojas commented 8 years ago

@feross, will publish a new version of esformatter-jsx soon. For now I published that branch as fresh-falafel until @substack has time to review/merge substack/node-falafel#51

royriojas commented 8 years ago

published esformatter-jsx@2.3.11 that doesn't use dependencies from github

feross commented 8 years ago

Nice, thanks!