Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add :npm-deps to the JavaScript Modules guide #80

Open
anmonteiro opened this issue May 31, 2017 · 3 comments
Open

Add :npm-deps to the JavaScript Modules guide #80

anmonteiro opened this issue May 31, 2017 · 3 comments
Labels

Comments

@anmonteiro
Copy link
Contributor

One particular caveat to mention is that, in some versions of NPM/Node, users need to have a package.json file present in the directory they're developing in or NPM will install modules to a parent directory.

@swannodette
Copy link
Member

Yes please!

@Deraen
Copy link
Contributor

Deraen commented Dec 21, 2017

We are also missing a mention that libraries can use :npm-deps in deps.cljs.

@swannodette
Copy link
Member

We should circle back and think about what we want to say about JavaScript modules post :target :bundle. While I think we should maintain our JS module support especially ES6 - we should think about how to make this really become an edgecase. I'm warming up to a defclass macro, there's a ticket in JIRA bout this - I think this one are where you have no recourse but to write JS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants