Hyperapp: A broken `main` ?

Created on 16 Nov 2020  路  3Comments  路  Source: jorgebucaran/hyperapp

Shouldn't the main prop in package.json be index.js instead of hyperapp.js ?

https://github.com/jorgebucaran/hyperapp/blob/8e941b1cf37cb5af9032e1aba635229d214b5bb8/package.json#L6

Most helpful comment

@icylace main: index.js makes sense, main: hyperapp.js, doesn't. Let's change it.

All 3 comments

@icylace Maybe. I'm not really sure right now. hyperapp.js is generated in build time and contains Hyperapp as an IIFE. But, why would that be the main entry to the module?

What I can say is that as I continue to work with my Hyperapp fork in my own projects as a way of stress-testing my Hyperapp type definitions, I've had to use index.js as the main so that Snowpack can use my fork properly.

In my projects that use my fork, I'm using a direct Github reference to my fork in package.json, so maybe that's significant ?

@icylace main: index.js makes sense, main: hyperapp.js, doesn't. Let's change it.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

jorgebucaran picture jorgebucaran  路  4Comments

SkaterDad picture SkaterDad  路  3Comments

dmitrykurmanov picture dmitrykurmanov  路  4Comments

jorgebucaran picture jorgebucaran  路  3Comments

dmitrykurmanov picture dmitrykurmanov  路  3Comments