-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[Due for payment 2025-03-17] Update verifyHybridApp workflow message to mention HybridApp repo patches check #57884
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Triggered auto assignment to @isabelastisser ( |
The PR has been opened for the review! |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.10-6 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2025-03-17. 🎊 For reference, here are some details about the assignees on this issue:
|
@mountiny @isabelastisser @VickyStash The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button] |
No need for checklists or payments, this was just DX improvement |
Uh oh!
There was an error while loading. Please reload this page.
Proposal: Update verifyHybridApp workflow message to mention HybridApp repo patches check
Background
For every E/App PR that is created from a fork and updates native code and/or dependencies the verifyHybridApp workflow triggers a warning to be shown:
Problem
When developers update dependencies in E/App, they may modify a library patched in the HybridApp repository.
It can lead to compatibility issues, which prevent the app from functioning correctly. Developers are not automatically alerted to this risk, which can lead to unnoticed breakages.
Solution
Update the warning message triggered by the verifyHybridApp workflow to ask the developer to check if patches inside the HybridApp repo need to be updated.
Issue Owner
Current Issue Owner: @isabelastisserThe text was updated successfully, but these errors were encountered: