Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add "main" field to package.json #2883

Closed
tientq64 opened this issue Sep 25, 2023 · 1 comment
Closed

Add "main" field to package.json #2883

tientq64 opened this issue Sep 25, 2023 · 1 comment
Assignees
@dead-claudia
Labels
Type: Enhancement For any feature request or suggestion that isn't a bug fix

Comments

@tientq64
Copy link

Description

Please add the "main" field to package.json, I see there is a "pkg" field in there but then it only supports "unpkg.com" CDN. I use "jsdelivr.net", using the "main" field will support both.

Why

Supports "jsdelivr.net" CDN.

Possible Implementation

Add the "main" field to package.json
@tientq64 tientq64 added the Type: Enhancement For any feature request or suggestion that isn't a bug fix label Sep 25, 2023
@dead-claudia
Copy link
Member

Resolved using their jsdelivr package key. I don't want to override main, as I want to reserve that for a non-minified entry point.

dead-claudia added a commit to dead-claudia/mithril.js that referenced this issue Sep 23, 2024
@dead-claudia
Add support for jsDelivr CDN
8673cd9 
Fixes MithrilJS#2883, but by using a special case.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dead-claudia dead-claudia

Labels
Type: Enhancement For any feature request or suggestion that isn't a bug fix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dead-claudia @tientq64