Skip to content

use the napiVersion and skipTest primitives for porting test_cannot_run_js#3

Merged
kraenhansen merged 1 commit intokraenhansen:feat/port-test-cannot-run-jsfrom
bavulapati:feat/port-test-cannot-run-js
Apr 25, 2026
Merged

use the napiVersion and skipTest primitives for porting test_cannot_run_js#3
kraenhansen merged 1 commit intokraenhansen:feat/port-test-cannot-run-jsfrom
bavulapati:feat/port-test-cannot-run-js

Conversation

@bavulapati
Copy link
Copy Markdown

@bavulapati bavulapati commented Apr 11, 2026

Address nodejs#32 (comment)

The changes look complex as the target branch needs a rebase. Once rebased, it is a simple change.

@bavulapati bavulapati changed the title Feat/port test cannot run js use the napiVersion and skipTest primitives Apr 13, 2026
@bavulapati bavulapati changed the title use the napiVersion and skipTest primitives use the napiVersion and skipTest primitives for porting test_cannot_run_js Apr 13, 2026
@kraenhansen kraenhansen force-pushed the feat/port-test-cannot-run-js branch from a0f9aee to a590d2c Compare April 25, 2026 20:51
Signed-off-by: Balakrishna Avulapati <ba@bavulapati.com>
@kraenhansen kraenhansen force-pushed the feat/port-test-cannot-run-js branch from f267a30 to 264808e Compare April 25, 2026 21:15
@kraenhansen kraenhansen merged commit b988c69 into kraenhansen:feat/port-test-cannot-run-js Apr 25, 2026
13 checks passed
@kraenhansen
Copy link
Copy Markdown
Owner

And thanks here as well 🙂

@bavulapati bavulapati deleted the feat/port-test-cannot-run-js branch April 26, 2026 06:32
@bavulapati
Copy link
Copy Markdown
Author

I think I missed a detail here. My bad. I left a comment on nodejs#28

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.

2 participants