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

Add Javascript test setup #29

Closed
wants to merge 6 commits into from
Closed

Conversation

ghost
Copy link

@ghost ghost commented Aug 12, 2024

Add a basic set up for running revive-wasm tests. This is intended as an addition the the revive on wasm work being done in smiasojed/revive.git.

@smiasojed smiasojed mentioned this pull request Aug 22, 2024
@xermicus
Copy link
Member

@wpt967 what is the exact status of this PR? We should avoid letting it rot for too long and try getting it merged soon. After #66, we can investigate if the in-browser memory limitations are still bad enough. But it doesn't block this work from being finished. Can we bring this PR to a state where the compiler works fine in node where we there are basically no resource constraint? There was also #26. IIRC the only thing missing there was proper testing on the CI.

@ghost
Copy link
Author

ghost commented Oct 14, 2024

Defunct, closed.

@ghost ghost closed this Oct 14, 2024
This pull request was closed.
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