aurelia / bundler

A library for bundling JavaScript, HTML and CSS for use with SystemJS.
MIT License
37 stars 25 forks source link

Bundler not tracing index.js when requiring a folder #173

Closed fragsalat closed 6 years ago

fragsalat commented 6 years ago

As I think this was by intent... I'm submitting a feature request

Please tell us about your environment:

Current behavior: A package like @material-ui/code requires folders which contains a index.js and those are not recognized by the bundler.

/folder1
  file1.js
  /folder2
    index.js

file1.js

const exportOfIndexJs = require('/folder1/folder2');

Expected/desired behavior:

bigopon commented 6 years ago

@huochunpeng should this be resolved with auto tracing ?

3cp commented 6 years ago

Yes. It is part of nodejs compatibility I implemented there.

fragsalat commented 6 years ago

What does this mean? Should I just update all packages and it should work? Will it be fixed with a future release or should it already work and I made something wrong?

3cp commented 6 years ago

It means it will be fixed after aurelia/cli#862 got released.

fragsalat commented 6 years ago

Muaha that's great :) Thanks men :)

Alexander-Taran commented 6 years ago

Can be closed

fragsalat commented 6 years ago

:+1: