-
Notifications
You must be signed in to change notification settings - Fork 1
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
[7.4 only] LPS-122456 - Remove {append, buildFragment, enterDocument} usages of metal-dom #468
Conversation
To conserve resources, the PR Tester does not automatically run for every pull. If your code changes were already tested in another pull, reference that pull in this pull so the test results can be analyzed. If your pull was never tested, comment "ci:test" to run the PR Tester for this pull. |
ci:test:sf |
ci:test:relevant |
✔️ ci:test:sf - 1 out of 1 jobs passed in 3 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: pr-463 1 Successful Jobs:For more details click here. |
Jenkins Build:test-portal-source-format#3937 |
❌ ci:test:stable - 6 out of 9 jobs passed❌ ci:test:relevant - 13 out of 30 jobs passed in 2 hours 43 minutesClick here for more details.Base Branch:Branch Name: master Copied in Private Modules Branch:Branch Name: master-private ci:test:stable - 6 out of 9 jobs PASSED3 Failed Jobs:6 Successful Jobs:
ci:test:relevant - 13 out of 30 jobs PASSED17 Failed Jobs:
13 Successful Jobs:
For more details click here.Failures unique to this pull:
|
Jenkins Build:test-portal-acceptance-pullrequest(master)#6507 |
@john-co I also can't seem to figure out why this one is breaking as well... I looked into the poshi report, but locally it seems fine again, so I'm not sure if this is a flakey test or something else. |
ci:test:bundle |
❌ ci:test:bundle - 0 out of 1 jobs passed in 56 minutesClick here for more details.Base Branch:Branch Name: master ci:test:bundle - 0 out of 1 jobs PASSED1 Failed Jobs:For more details click here.Failures unique to this pull:
For upstream results, click here.Test bundle downloads:
|
There seems to have been an unrelated issue prior to running some functional tests...can you fix the merge conflict and run the tests again? |
b656d5d
to
1d1e8da
Compare
ci:test:relevant |
4d84140
to
7fa9669
Compare
ci:test:sf |
ci:test:relevant |
✔️ ci:test:sf - 1 out of 1 jobs passed in 5 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: pr-463 1 Successful Jobs:For more details click here. |
Jenkins Build:test-portal-source-format#3460 |
ci:stop |
This PR is gonna be dependent on the jest mocks provided with the new npm-scripts at #506 |
❌ ci:test:stable - 6 out of 9 jobs passed❌ ci:test:relevant - 14 out of 30 jobs passed in 3 hours 36 minutesClick here for more details.Base Branch:Branch Name: master Copied in Private Modules Branch:Branch Name: master-private ci:test:stable - 6 out of 9 jobs PASSED3 Failed Jobs:6 Successful Jobs:
ci:test:relevant - 14 out of 30 jobs PASSED16 Failed Jobs:
14 Successful Jobs:
For more details click here.Failures unique to this pull:
|
Jenkins Build:test-portal-acceptance-pullrequest(master)#7252 |
I'm going to close this and rebase on top of #529 to remove the Senna changes and maybe even remove them completely as to try to get this unblocked! |
See #530 |
follow up from #463