Skip to content

fix(ext/node): handle cargo features during publish#32636

Merged
bartlomieju merged 2 commits intodenoland:mainfrom
bartlomieju:fix_node_fs_publish
Mar 11, 2026
Merged

fix(ext/node): handle cargo features during publish#32636
bartlomieju merged 2 commits intodenoland:mainfrom
bartlomieju:fix_node_fs_publish

Conversation

@bartlomieju
Copy link
Copy Markdown
Member

hot fix for v2.7.5 release that broke during cargo publish step.

Copy link
Copy Markdown
Contributor

@kajukitli kajukitli left a comment

Choose a reason for hiding this comment

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

lgtm — macro handles the conditional spawn_blocking based on feature flag cleanly

Copy link
Copy Markdown
Contributor

@marvinhagemeister marvinhagemeister left a comment

Choose a reason for hiding this comment

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

LGTM

@bartlomieju bartlomieju merged commit 9bb8552 into denoland:main Mar 11, 2026
112 checks passed
@bartlomieju bartlomieju deleted the fix_node_fs_publish branch March 11, 2026 12:39
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.

3 participants