Skip to content

Revive CI#2

Merged
ofalvai merged 1 commit into
masterfrom
push-qqxnwprtwqqn
Jul 14, 2026
Merged

Revive CI#2
ofalvai merged 1 commit into
masterfrom
push-qqxnwprtwqqn

Conversation

@ofalvai

@ofalvai ofalvai commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

History and background context:

Changes

  • Delete Azure CI pipeline configs, but migrate the actual CI logic to GHA
  • Set up a quick lint job that runs first and doesn't involve slow sample app builds, emulators/simulators.
  • Make the Android and iOS test jobs actually work:
    • Android: this was already in a good shape. We use the de-facto https://github.com/ReactiveCircus/android-emulator-runner instead of the shell script tweaks in the old Azure config.
    • iOS: had to fix one known issue between RN 0.82 and Xcode 26.4+, and also revert one change that I'm 90% sure caused a real regression. It's the Codepush.m workaround introduced in this commit. Dispatching to another thread definitely broke the revert logic, it resulted in 6 broken iOS test scenarios. Meanwhile, I couldn't reproduce that race condition mentioned in the workaround, so I think we better revert that workaround, it at least fixed the revert logic. Even if that race condition is real, it might be already fixed upstream (we are testing with RN 0.82 at the moment, latest stable version is 0.86)
  • Set up tool version pinning via Mise and mise.toml.
  • Runners (runs-on): set up some org-wide hosted large runners. On the Linux side, these are needed because of nested virtualization, and the default macOS runners are too weak for our stuff.

Out of scope

  • Optimize CI performance or reliability: later
  • Revisit Node/JDK/etc tool version, do upgrades: later

@ofalvai ofalvai force-pushed the push-qqxnwprtwqqn branch 2 times, most recently from 3aaaffa to acf9b4e Compare July 9, 2026 11:38
@ofalvai ofalvai changed the title Clean up GHA config Clean up historical CI config Jul 9, 2026
@ofalvai ofalvai force-pushed the push-qqxnwprtwqqn branch 10 times, most recently from f0e41f3 to 46b2033 Compare July 13, 2026 10:43
# Tests need `emulator` to be in $PATH.
PATH: /usr/local/lib/android/sdk/emulator:/usr/local/lib/android/sdk/platform-tools:${{ env.PATH }}
with:
api-level: 27

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

TODO: upgrade

@ofalvai ofalvai force-pushed the push-qqxnwprtwqqn branch from 46b2033 to ae44832 Compare July 14, 2026 08:09
@ofalvai ofalvai changed the title Clean up historical CI config Revive CI Jul 14, 2026
@ofalvai ofalvai marked this pull request as ready for review July 14, 2026 08:29
@ofalvai ofalvai mentioned this pull request Jul 14, 2026
@ofalvai ofalvai merged commit 4d3e2a2 into master Jul 14, 2026
3 checks passed
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