Skip to content

[NFC] revert msvc exclusion in wasm-type-printing#8706

Closed
Changqing-JING wants to merge 2 commits into
WebAssembly:mainfrom
Changqing-JING:revert-msvc
Closed

[NFC] revert msvc exclusion in wasm-type-printing#8706
Changqing-JING wants to merge 2 commits into
WebAssembly:mainfrom
Changqing-JING:revert-msvc

Conversation

@Changqing-JING

@Changqing-JING Changqing-JING commented May 15, 2026

Copy link
Copy Markdown
Contributor

@Changqing-JING
Changqing-JING requested a review from a team as a code owner May 15, 2026 04:37
@Changqing-JING
Changqing-JING requested review from kripken and removed request for a team May 15, 2026 04:37
@Changqing-JING
Changqing-JING marked this pull request as draft May 15, 2026 04:37
@kripken

kripken commented May 15, 2026

Copy link
Copy Markdown
Member

From the error on CI, it looks like this fix did not reach it?

@Changqing-JING

Copy link
Copy Markdown
Contributor Author

Yes, you are correct. I just checked GitHub release notes. The currently windows-latest runner are still using visual studio 2022. And the migration will be done next month. So I just leave this PR as draft and we can check it later.
https://github.blog/changelog/2026-05-14-github-actions-upcoming-image-migrations/

@Changqing-JING

Copy link
Copy Markdown
Contributor Author

This PR is not needed anymore since the target code is refactored

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