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

docs: updated with resolve settings #688

Closed
wants to merge 1 commit into from

Conversation

rajuahmmed
Copy link

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

Just doc update with resolve extensions setting. I think doc need to updated because of every day #556 thread is getting bigger and bigger. It's better to update into doc so everyone can fix there issue.

Breaking Changes

There is no breaking changes.

Additional Info

@jsf-clabot
Copy link

jsf-clabot commented May 27, 2019

CLA assistant check
All committers have signed the CLA.

@rajuahmmed rajuahmmed changed the title doc: updated with resolve settings docs: updated with resolve settings May 27, 2019
@yisding
Copy link

yisding commented Jun 20, 2019

@import "base" will still fail, with these additional options in certain conditions.

Adding:
mainFiles: []

will prevent it from trying to import base/index.js

@alexander-akait
Copy link
Member

Thanks for the PR, issue was solved in latest release (check you use latest webpack version), no need put this here

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

4 participants