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

feat!: Esm and rollup bump #308

Merged
merged 7 commits into from
Sep 6, 2023
Merged

feat!: Esm and rollup bump #308

merged 7 commits into from
Sep 6, 2023

Conversation

skjalgepalg
Copy link
Collaborator

  • Change package to module and replace top-level "browser" with "exports".
  • Removed .github folder from npm bundle
  • Bump rollup to v3.28.1 changing how we import info from package.json to comply with standard/eslint parsing.

While generated files remain the same and we validate with publint, changes to exports in package.json should be pushed as a major/breaking change.

@skjalgepalg skjalgepalg added the enhancement New feature or request label Aug 25, 2023
@skjalgepalg skjalgepalg requested a review from klizter August 25, 2023 06:32
@skjalgepalg skjalgepalg self-assigned this Aug 25, 2023
@skjalgepalg skjalgepalg merged commit 1a53608 into master Sep 6, 2023
@skjalgepalg skjalgepalg deleted the esm-rollup-bump branch September 6, 2023 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@klizter klizter klizter approved these changes

Assignees

@skjalgepalg skjalgepalg

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@skjalgepalg @klizter