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

feat(node-resolve): add rootDir option #98

Merged
merged 7 commits into from
Dec 25, 2019

Conversation

LarsDenBakker
Copy link
Contributor

@LarsDenBakker LarsDenBakker commented Dec 20, 2019

Rollup Plugin Name: node-resolve

This PR contains:

  • bugfix
  • feature
  • refactor
  • documentation
  • other

Are tests included?

  • yes (bugfixes and features will not be merged without tests)
  • no

Breaking Changes?

  • yes (breaking changes will not be merged unless absolutely necessary)
  • no

List any relevant issue numbers: fixes #86

Description

Introduces a rootDir option, which is used to resolve modules which need to imported from root. Right now these are entrypoint modules and deduped modules. The default value is process.cwd(), which is also what it used right now making this a non-breaking change.

@shellscape
Copy link
Collaborator

Thanks for creating this. It looks like the ESLint TS plugin is on the fritz again. I'll try to fix that in the next day or so.

packages/node-resolve/README.md Outdated Show resolved Hide resolved
packages/node-resolve/test/root-dir.js Outdated Show resolved Hide resolved
Copy link
Member

@NotWoods NotWoods left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@shellscape
Copy link
Collaborator

shellscape commented Dec 25, 2019

Anyone have a clue as to why Github Actions wouldn't be running for this PR and @LarsDenBakker other PRs? Figured it out.

@shellscape
Copy link
Collaborator

thanks!

@shellscape shellscape merged commit df6835e into rollup:master Dec 25, 2019
LarsDenBakker added a commit to LarsDenBakker/plugins that referenced this pull request Sep 12, 2020
* feat(node-resolve): add rootDir option

* chore: update .eslintignore for new fixtures

* test: add missing snapshots

* Update packages/node-resolve/README.md

Co-Authored-By: Tiger Oakes <[email protected]>

* test(node-resolve): remove test.only

Co-authored-by: Andrew Powell <[email protected]>
Co-authored-by: Tiger Oakes <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

dedupe leads to errors in a monorepo with hoisted dependencies
3 participants