Skip to content
This repository has been archived by the owner on Jun 1, 2022. It is now read-only.

File dist/commonjs/index.js does not exist #157

Closed
adriatic opened this issue Sep 2, 2015 · 5 comments
Closed

File dist/commonjs/index.js does not exist #157

adriatic opened this issue Sep 2, 2015 · 5 comments

Comments

@adriatic
Copy link
Contributor

adriatic commented Sep 2, 2015

There is a statement in package.json at line 16:

"main": "dist/commonjs/index.js"

No such file in the dist folder.

image

@adriatic
Copy link
Contributor Author

adriatic commented Sep 2, 2015

I am just guessing that this is a leftover from previous process of starting the app and that the app is correctly launched nevertheless because of the different jspm defaults in place now.

@plwalters
Copy link
Contributor

I think this is an artifact from the other repositories, probably from using a similar skeleton for repositories. Should be unneeded because we aren't installing skeleton-navigation through npm / jspm, did you want to PR a clean up?

@adriatic
Copy link
Contributor Author

adriatic commented Sep 3, 2015

Not sure that I understand your statement ''we aren't installing skeleton-navigation through npm / jspm".

What about the highlighted items in this screenshot?

image

@EisenbergEffect
Copy link
Contributor

That section of the npm file only applies if skeleton-navigation were itself an npm package that was installed into another project. While that being there is harmless, we should probably just remove it.

@adriatic
Copy link
Contributor Author

adriatic commented Sep 3, 2015

Makes perfect sense now (one of the very few jspm <--> npm interactions related issues that haunt me for weeks :-( )

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants