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

webpack should be direct dependency #137

Closed
wants to merge 2 commits into from

Conversation

rashidul0405
Copy link

lerna couldn't resolve webpack dependency.

We've lerna.js setup.

  • A project use webpack@4.
  • Another project use webpack@3
    When we run dotenv-webpack from webpack@3 project's scope, it always resolves to webpack@4 as webpack@4 is on the global scope.

lerna couldn't resolve webpack dependency.

We've lerna.js setup.
- A project use webpack@4.
- Another project use webpack@3
When we run dotenv-webpack from webpack@3 project's scope, it always resolves to webpack@4 as webpack@4 is on the global scope.
@rashidul0405
Copy link
Author

we are using version 1.5.4 due to compatibility with webpack@3, can this fix be included with 1.5.4 release, please?

@rashidul0405
Copy link
Author

travis-ci build disabled: Build type disabled via repository settings (pull_request).

@mrsteele
Copy link
Owner

@rashidul0405 Thanks for the PR, however I am a little confused.

We have a "dev" dependency on our personal development to include webpack for the purposes of our unit testing. So in your environment you shouldn't need the dev dependency.

We also have the peer dependency here: https://github.com/mrsteele/dotenv-webpack/pull/137/files#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R42

This should satisfy the conflicting dependency issue you see as they are both covered.

@rashidul0405
Copy link
Author

@mrsteele Thank you very much for your reply. It could be related to https://github.com/lerna/lerna. I'll explore more and will get back to you.

@mrsteele
Copy link
Owner

I'm going to close this as I think this has gone stale, but if you think it should be reopened/reconsidered, feel free to reopen.

@mrsteele mrsteele closed this Oct 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants