css-ratiocinator icon indicating copy to clipboard operation
css-ratiocinator copied to clipboard

don't include vendor code in repsotory

Open losingkeys opened this issue 12 years ago • 2 comments

Shouldn't vendor code (e.g. the stuff at vendor/phantom-jasmine/) be a submodule (if you just want the code) or listed in package.json (if you want it installable via npm)?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

losingkeys avatar Mar 01 '13 15:03 losingkeys

Good point. I wonder how this will work in csstrashman. That site includes ratiocinator as a submodule which heroku pulls in on deploy. Will a submodule inside a submodule work OK?

I don't know much about npm, perhaps it's the right solution for this dependency stuff. While I'm thinking about it, the functions in lib/css.js might be exposed better in an npm-style export than the way they are currently exposed by being added to the window object.

begriffs avatar Mar 01 '13 16:03 begriffs

I don't understand, why do you want to nest them?

losingkeys avatar Mar 01 '13 17:03 losingkeys