Skip to content
This repository has been archived by the owner on Jan 6, 2025. It is now read-only.

Incorrect imports in built sources #186

Closed
nlepage opened this issue Dec 26, 2017 · 0 comments · Fixed by #187
Closed

Incorrect imports in built sources #186

nlepage opened this issue Dec 26, 2017 · 0 comments · Fixed by #187
Assignees
Labels
Milestone

Comments

@nlepage
Copy link
Member

nlepage commented Dec 26, 2017

Description

Sources generated with babel contain incorrect import paths.

Only for bugs 🐛

Version : 1.0.0-rc.1

Expected behavior

...

Actual behavior

Module not found: Error: Can't resolve '../../../src/core/set' in './node_modules/immutadot/seq/core'
 @ ./node_modules/immutadot/seq/core/set.js 3:4-79
 @ ./node_modules/immutadot/seq/core/index.js
Module not found: Error: Can't resolve '../../../src/seq/ChainWrapper' in './node_modules/immutadot/seq/array'
 @ ./node_modules/immutadot/seq/array/push.js 3:4-67
 @ ./node_modules/immutadot/seq/array/index.js
Module not found: Error: Can't resolve 'core/add' in './node_modules/immutadot/seq/lang'
 @ ./node_modules/immutadot/seq/lang/add.js 3:4-66
 @ ./node_modules/immutadot/seq/lang/index.js
Module not found: Error: Can't resolve 'seq/core' in './node_modules/immutadot/util'
 @ ./node_modules/immutadot/util/protect.js 3:4-88
 @ ./node_modules/immutadot/util/index.js
@nlepage nlepage added this to the 1.0.0-rc.2 milestone Dec 26, 2017
@nlepage nlepage self-assigned this Dec 26, 2017
nlepage added a commit that referenced this issue Dec 26, 2017
* 🔧 Add ./generated to the prod env of babel module-resolver

* 🐛 fix generated sources for seq

* 🐛 Use a temporary merged source directory fix #186
frinyvonnick pushed a commit that referenced this issue Jan 3, 2018
* 🔧 Add ./generated to the prod env of babel module-resolver

* 🐛 fix generated sources for seq

* 🐛 Use a temporary merged source directory fix #186
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant