[Due for payment 2025-02-13] Improve checkout times in verifyHybridAppBuild workflow

[roryabraham]

#55328 was merged just before I got my review in. Creating this to address some follow-up.

[roryabraham]

Looks like @AndrewGable has a draft up here: #55807

But as discussed in 1:1's today, this workflow won't work for PRs from forks. We may need to step back and reevaluate the approach. I think the best approach we landed on today was a flow like this:

We could build a check that looks for a passing AdHoc build using that PR. It passes if someone has manually run an AdHoc build that has passed, and it fails if not. But the AdHoc build has to be run manually after an internal engineer verifies the code is not malicious

[AndrewGable]

Do you think we should build for non-forks and leave a comment for the forks?

[roryabraham]

Yeah, that sounds good. We could also do more than just leave a comment for forks - i.e: look for a passing AdHoc build workflow for the PR, and fail the check if it's not found

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.94-25 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• [No QA]Followups for HybridApp building on main test #55807

If no regressions arise, payment will be issued on 2025-02-13. 🎊

[roryabraham]

This is done, no payments due, closing it out