Skip to content

feat!: include dual cjs/esm in package exports - #447

Merged
userquin merged 3 commits into
mainfrom
userquin/fix-package-exports
Nov 24, 2023
Merged

feat!: include dual cjs/esm in package exports#447
userquin merged 3 commits into
mainfrom
userquin/fix-package-exports

Conversation

@userquin

@userquin userquin commented Nov 23, 2023

Copy link
Copy Markdown
Member

Description

This PR include:

  • removes the postinstall script, using default output from tsup seem to be ok
  • split package exports in require/import
  • include publint

Linked Issues

Additional context

Current types:

imagen

With this PR:

imagen

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@socket-security

Copy link
Copy Markdown

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
publint 0.2.5 None +1 96.7 kB bluwy

@userquin
userquin marked this pull request as draft November 23, 2023 21:30
@userquin

This comment was marked as resolved.

@userquin
userquin marked this pull request as ready for review November 23, 2023 22:12
@userquin
userquin requested a review from antfu November 23, 2023 22:15
@userquin userquin changed the title fix: include dual cjs/esm in package exports feat!: include dual cjs/esm in package exports Nov 23, 2023
@userquin
userquin merged commit b43b317 into main Nov 24, 2023
@userquin
userquin deleted the userquin/fix-package-exports branch November 24, 2023 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant