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 spec tests for the custom-page-sizes proposal #23

Merged
merged 2 commits into from
Jun 12, 2024
Merged

Conversation

fitzgen
Copy link
Collaborator

@fitzgen fitzgen commented Jun 10, 2024

These were written while implementing the proposal in Wasmtime: bytecodealliance/wasmtime#8763

These were written while implementing the proposal in Wasmtime:
bytecodealliance/wasmtime#8763
@fitzgen
Copy link
Collaborator Author

fitzgen commented Jun 11, 2024

@rossberg or @conrad-watt, is there an easy way to mark certain spec tests as ignored by the spec interpreter temporarily so that CI doesn't fail indefinitely until I implement this proposal for the spec interpreter? I didn't find anything in run.py or the makefile but quite possible I am just missing something.

@rossberg
Copy link
Member

I don't think we currently have a means of turning off individual tests. The easiest thing you can do right now is just commenting out the entire test step in the GH workflow for the interpreter.

@fitzgen
Copy link
Collaborator Author

fitzgen commented Jun 12, 2024

@rossberg makes sense, thanks!

They can be re-enabled once the spec interpreter gains support for the
custom-page-sizes proposal.
@fitzgen fitzgen merged commit 1a81043 into main Jun 12, 2024
5 checks passed
@fitzgen fitzgen deleted the add-spec-tests branch June 12, 2024 17: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.

2 participants