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

chore(node): improve node/binding structure #550

Merged
merged 10 commits into from Mar 11, 2024

Conversation

hyf0
Copy link
Member

@hyf0 hyf0 commented Mar 11, 2024

Description

Closes #545.

I see "@napi-rs/cli": "^3.0.0-alpha.43" is smart enough to determine whether to rebuild or not if changes don't contain rust files.

Closes #432.

Test Plan


Copy link
Member Author

hyf0 commented Mar 11, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hyf0 and the rest of your teammates on Graphite Graphite

Copy link

netlify bot commented Mar 11, 2024

Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit 2434ba5
🔍 Latest deploy log https://app.netlify.com/sites/rolldown-rs/deploys/65ef2c1c3f71d300084fbea5

@hyf0 hyf0 changed the title Closes #545 chore(node): improve node/binding structure Mar 11, 2024
@hyf0 hyf0 force-pushed the chore_node_improve_the_node/binding_structure branch from 1fe69a5 to ec19430 Compare March 11, 2024 14:15
@hyf0 hyf0 force-pushed the chore_node_improve_the_node/binding_structure branch from f01ecc0 to aa99b39 Compare March 11, 2024 15:08
@hyf0 hyf0 marked this pull request as ready for review March 11, 2024 16:07
@hyf0
Copy link
Member Author

hyf0 commented Mar 11, 2024

Since this is a big folder structure change and the ci run successfully, let's merge it first and fix possible issue on top of it.

@hyf0 hyf0 merged commit 23b0cc0 into main Mar 11, 2024
12 of 14 checks passed
@hyf0 hyf0 deleted the chore_node_improve_the_node/binding_structure branch March 11, 2024 16:12
]
},
"dependencies": {
"prettier": "^3.2.5"
Copy link
Member

Choose a reason for hiding this comment

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

This is by mistake right?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah.

@@ -0,0 +1,76 @@
{
"name": "rolldown",
"version": "0.0.1",
Copy link
Member

Choose a reason for hiding this comment

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

The package previously has versions published up to 0.3.0 and 0.4.x-alpha, we should start at 0.5.0 just to be safe.

@yyx990803
Copy link
Member

@hyf0 found some small issues. Pinging in case the inline comments get lost.

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