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.

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

feat!: update to rollup v4 #1018

Merged
merged 1 commit into from
Oct 16, 2023
Merged

feat!: update to rollup v4 #1018

merged 1 commit into from
Oct 16, 2023

Conversation

sapphi-red
Copy link
Contributor

  • update rollup plugins for rollup v4 support
  • update rollup to v4
  • update minimum node version to 18 (rollup v4 supports Node 18+)

@codesandbox
Copy link

codesandbox bot commented Oct 8, 2023

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@vercel
Copy link

vercel bot commented Oct 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tsup ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 8, 2023 11:21am

Comment on lines -79 to +78
"typescript": ">=4.1.0"
"typescript": ">=4.5.0"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -25,7 +24,6 @@ export const treeShakingPlugin = ({
const bundle = await rollup({
input: [info.path],
plugins: [
hashbang(),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sxzz sxzz merged commit 438ec47 into egoist:dev Oct 16, 2023
7 checks passed
@sapphi-red sapphi-red deleted the feat/update-rollup4 branch October 16, 2023 12:51
Copy link
Contributor

🎉 This PR is included in version 7.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sapphi-red
Copy link
Contributor Author

@egoist @sxzz This PR includes a breaking change. Was it fine to release in a minor?

@sxzz
Copy link
Collaborator

sxzz commented Nov 18, 2023

I think the minor is good. Rollup is used internally in tsup, and didn't exposed for tsup user.

@sapphi-red
Copy link
Contributor Author

Upgrading rollup itself is not a breaking change. But this PR upgrades the minimum supported Node.js version and bumps peerDep TypeScript version. Both of them are breaking changes.

@sxzz
Copy link
Collaborator

sxzz commented Nov 18, 2023

Oh you're right.

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

Successfully merging this pull request may close these issues.

2 participants