Skip to content
This repository was archived by the owner on Aug 4, 2021. It is now read-only.

Use correct context when manually resolving#370

Merged
lukastaegert merged 1 commit intomasterfrom
use-resolve-context
Feb 23, 2019
Merged

Use correct context when manually resolving#370
lukastaegert merged 1 commit intomasterfrom
use-resolve-context

Conversation

@lukastaegert
Copy link
Member

@lukastaegert lukastaegert commented Jan 23, 2019

Rollup is manually parsing the plugins array in order to resolve some dependencies. This needs to use the same context as rollup does.

This fixes rollup/rollup-plugin-node-resolve#196

Also update dependencies.

@simonhaenisch
Copy link

Can we move forward with this?

@simonhaenisch
Copy link

@lukastaegert any plans to release this soon?

@lukastaegert lukastaegert merged commit d9a2f08 into master Feb 23, 2019
@lukastaegert lukastaegert deleted the use-resolve-context branch February 23, 2019 13:23
@lukastaegert
Copy link
Member Author

Sorry, got side-tracked by other things. Is released now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot read property 'warn' of undefined when preferBuiltins is not set

2 participants