-
Notifications
You must be signed in to change notification settings - Fork 138
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
COMMERCE-2228 AddToCartButton is aware of product removal in cart #1601
COMMERCE-2228 AddToCartButton is aware of product removal in cart #1601
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. |
221752a
to
d078d1c
Compare
ci:test |
Validation FAILED.Build Time: 21 minutes 39 seconds 475 ms Base Branch:Branch Name: 7.1.x Task Summary:
For full console, click here. |
d078d1c
to
eb296cc
Compare
ci:test |
Validation PASSED. Running batch tests.Build Time: 20 minutes 8 seconds 707 ms Base Branch:Branch Name: 7.1.x Task Summary:
For full console, click here. |
❌ ci:test:default - 3 out of 13 jobs passed in 1 hour 46 minutes 246 msClick here for more details.Base Branch:Branch Name: 7.1.x ci:test:default - 3 out of 13 jobs PASSED10 Failed Jobs:
For more details click here.Failures (upstream comparison is not available):
|
Error caused by COMMERCE-2283. Working on a fix. |
Hey @ethanbustad ! How does your fix relate with this PR? Or you mean the tests randomly failing? |
@gianmarcobrunialti the tests are failing because private modules aren't resolving -- they have been behind dependency-wise for a while, and I guess a CI change must have started deploying them, so now tests are failing across all PRs. So we just need to fix the dependency issues in the private modules, and this failure should go away -- i.e. the failure is not caused by your changes. |
Thanks for the explanation @ethanbustad ! |
eb296cc
to
0debf07
Compare
ci:test |
Validation PASSED. Running batch tests.Build Time: 24 minutes 57 seconds 431 ms Base Branch:Branch Name: 7.1.x Task Summary:
For full console, click here. |
❌ ci:test:default - 3 out of 13 jobs passed in 1 hour 50 minutes 10 seconds 758 msClick here for more details.Base Branch:Branch Name: 7.1.x ci:test:default - 3 out of 13 jobs PASSED10 Failed Jobs:
For more details click here.Failures (upstream comparison is not available):
|
ci:test |
Validation PASSED. Running batch tests.Build Time: 24 minutes 513 ms Base Branch:Branch Name: 7.1.x Task Summary:
For full console, click here. |
No description provided.