You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
That linked PR runs the full E/App test suite signals that the Onyx
15
+
change is safe. After this PR is merged and published to npm, swap the linked E/App PR to the pinned version
16
+
(e.g. "react-native-onyx": "3.0.72") and request a final review — it becomes the bump PR.
17
+
18
+
Paste the full Expensify/App PR URL below (e.g. https://github.com/Expensify/App/pull/12345):
19
+
-->
20
+
21
+
10
22
### Automated Tests
11
23
<!---
12
24
Most changes to Onyx should have accompanying tests. Describe the tests you added or if no tests were added an explanation about why one was not needed.
@@ -20,6 +32,7 @@ Each set of changes should be tested against the Expensify/App repo on all platf
20
32
### Author Checklist
21
33
22
34
-[ ] I linked the correct issue in the `### Related Issues` section above
35
+
-[ ] I linked the corresponding Expensify/App PR in the `### Linked E/App PR` section above, and verified this change against it (E/App CI passed and manual testing completed)
23
36
-[ ] I wrote clear testing steps that cover the changes made in this PR
24
37
-[ ] I added steps for local testing in the `Tests` section
25
38
-[ ] I tested this PR with a [High Traffic account](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#high-traffic-accounts) against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).
@@ -30025,6 +30026,13 @@ function validateChecklist(body) {
30025
30026
if (unchecked > 0) {
30026
30027
errors.push(`${unchecked} checklist item(s) are unchecked. All items must be checked before merging — including items that don't apply (check them and note why if needed).`);
errors.push('The "Linked E/App PR" section is empty. Every Onyx PR must link to a corresponding Expensify/App PR that pins this PR via git+https and runs the full E/App test suite.');
30032
+
}
30033
+
else if (!E_APP_PR_URL_PATTERN.test(linkedEAppPR)) {
30034
+
errors.push(`The "Linked E/App PR" section must contain a single Expensify/App PR URL (e.g. https://github.com/Expensify/App/pull/12345), found: "${linkedEAppPR}".`);
30035
+
}
30028
30036
// Section warnings
30029
30037
if (!getSectionContent(body, 'Automated Tests')) {
30030
30038
warnings.push('The "Automated Tests" section is empty. Please describe the automated tests you added, or explain why automated tests are not needed for this change.');
@@ -67,6 +68,13 @@ function validateChecklist(body: string): ValidationResult {
67
68
errors.push(`${unchecked} checklist item(s) are unchecked. All items must be checked before merging — including items that don't apply (check them and note why if needed).`);
errors.push('The "Linked E/App PR" section is empty. Every Onyx PR must link to a corresponding Expensify/App PR that pins this PR via git+https and runs the full E/App test suite.');
errors.push(`The "Linked E/App PR" section must contain a single Expensify/App PR URL (e.g. https://github.com/Expensify/App/pull/12345), found: "${linkedEAppPR}".`);
76
+
}
77
+
70
78
// Section warnings
71
79
if(!getSectionContent(body,'Automated Tests')){
72
80
warnings.push('The "Automated Tests" section is empty. Please describe the automated tests you added, or explain why automated tests are not needed for this change.');
0 commit comments