Skip to content

Re-enable removed testing onnx from CI #11071

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 19, 2025
Merged

Re-enable removed testing onnx from CI #11071

merged 1 commit into from
Jun 19, 2025

Conversation

TrumsZ
Copy link
Contributor

@TrumsZ TrumsZ commented Jun 19, 2025

This PR is raised to re-enable the testing of onnx in the CI according to the discussion in #10411.

@TrumsZ TrumsZ marked this pull request as ready for review June 19, 2025 01:56
@TrumsZ TrumsZ requested a review from a team as a code owner June 19, 2025 01:56
@TrumsZ TrumsZ requested review from cfallin and removed request for a team June 19, 2025 01:56
@TrumsZ TrumsZ mentioned this pull request Jun 19, 2025
Merged
abrown
abrown approved these changes Jun 19, 2025
View reviewed changes
Copy link
Member

@abrown abrown left a comment

Choose a reason for hiding this comment

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

Thanks for this PR; let's see if it works when I put it in the merge queue! The merge queue runs all of CI whereas by default a PR only runs a subset of the jobs (one can override this by adding prtest:full in the body of a commit message).

@abrown abrown added this pull request to the merge queue Jun 19, 2025
Merged via the queue into bytecodealliance:main with commit 5b01f1b Jun 19, 2025
41 checks passed
@yanwucai yanwucai mentioned this pull request Jun 23, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abrown abrown abrown approved these changes

@cfallin cfallin Awaiting requested review from cfallin cfallin is a code owner automatically assigned from bytecodealliance/wasmtime-default-reviewers

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TrumsZ @abrown