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

Using dotenv with Claudia #39

Closed
duboff opened this issue May 10, 2016 · 5 comments
Closed

Using dotenv with Claudia #39

duboff opened this issue May 10, 2016 · 5 comments
Labels

Comments

@duboff
Copy link

duboff commented May 10, 2016

Hi, previously I was able to use dotenv with Claudia by adding my .env file to the files property of the package.json. However, now it is ignored. I imagine this is because the file is in .gitignore. Can you please clarify if this is a desired behaviour?

I would think files property should dominate over .gitignore

@gojko
Copy link
Member

gojko commented May 10, 2016

files should override .gitignore, i'll check. is your .env explicitly included in the files section?

@gojko gojko added the bug label May 10, 2016
@gojko
Copy link
Member

gojko commented May 11, 2016

this is fixed in 1.2.7, now on NPM. if the files property is specified in package.json, Claudia now just ignores .gitignore/.npmignore. Please let me know if this solved your problem

@gojko gojko closed this as completed May 11, 2016
@duboff
Copy link
Author

duboff commented May 11, 2016

Confirm, everything works as expected. Thank you!

@jmike
Copy link

jmike commented Dec 19, 2016

Shouldn't the same apply to folders? Currently the "files" property has precedence over .gitignore / .npmignore for individual files, but not folders.

@gojko
Copy link
Member

gojko commented Dec 19, 2016

@jmike since 2.2.0 claudia just uses npm pack to produce the package, so check the NPM package.json docs if you need to exclude/include folders. you can test if it gets created correctly by just running npm pack directly

the only exception is node_modules, which gets re-installed, but you can force claudia to use your local copy by using --use-local-dependencies

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