Skip to content

ci: add Windows matrix to Android build#726

Merged
janicduplessis merged 5 commits into
mainfrom
@janic/windows-ci
May 15, 2026
Merged

ci: add Windows matrix to Android build#726
janicduplessis merged 5 commits into
mainfrom
@janic/windows-ci

Conversation

@janicduplessis
Copy link
Copy Markdown
Collaborator

Summary

Adds windows-latest to the Android build matrix so we can validate whether #691 is a problem in this library or in the reporter's environment.

The report is a CMake link failure when building react_codegen_safeareacontext.so on Windows with RN 0.82.1 Fabric: every missing symbol is from libc++_shared / libc++abi (std::__ndk1::basic_string, __cxa_*, operator new/delete, vtable for std::exception, …), which suggests the C++ stdlib is not being linked into the codegen library on Windows. Without a Windows machine it's not possible to tell whether this reproduces against our example app or is specific to the reporter's setup.

Both android-build (Paper) and android-build-fabric (Fabric) jobs now run on Ubuntu and Windows. defaults.run.shell: bash is set so ./gradlew works the same way on both runners — GitHub-hosted Windows runners ship Git Bash. fail-fast: false keeps the existing Ubuntu coverage green even if the Windows job is broken.

The example app currently pins react-native@^0.80.1 while the issue is reported against 0.82.1; if the Windows job passes here we'll still need to confirm against 0.82 separately, but a green Windows build on 0.80 + Fabric would already be a strong signal that the codegen CMakeLists is fine and the failure is environmental.

Test Plan

@janicduplessis janicduplessis marked this pull request as ready for review May 15, 2026 18:26
@janicduplessis janicduplessis merged commit 93b7a32 into main May 15, 2026
6 checks passed
@janicduplessis janicduplessis deleted the @janic/windows-ci branch May 15, 2026 18:51
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.

1 participant