Skip to content

Conversation

@fs-eire
Copy link
Contributor

@fs-eire fs-eire commented May 20, 2025

Description

revise WASM CI to run test as later step than publishing artifacts. This allows download the binary to diagnose test failures.

Motivation and Context

@fs-eire fs-eire requested a review from Copilot May 20, 2025 18:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR revises the WASM CI workflow by updating the test steps used in the build and test process.

  • Removed duplicate test steps for Node.js and browser that were present in the previously commented code.
  • Reintroduced updated test steps for Node.js and browser with new command flags in the workflow file.

@fs-eire fs-eire self-assigned this May 21, 2025
@fs-eire fs-eire requested a review from snnn May 21, 2025 16:29
@fs-eire fs-eire merged commit a612118 into microsoft:main May 29, 2025
84 of 87 checks passed
quic-ankus pushed a commit to CodeLinaro/onnxruntime that referenced this pull request Nov 25, 2025
### Description
<!-- Describe your changes. -->

revise WASM CI to run test as later step than publishing artifacts. This
allows download the binary to diagnose test failures.


### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
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