Skip to content
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

LPS-143397 Fix styles for template editor when publications is enabled #120656

Conversation

liferay-continuous-integration
Copy link
Collaborator

Forwarded from: liferay-echo#9002 (Took 1 ci:forward attempt in 1 hour 35 minutes)
Console

@victorg1991
@liferay-echo

✔️ ci:test:stable - 27 out of 27 jobs passed

✔️ ci:test:relevant - 190 out of 196 jobs passed in 1 hour 34 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 22411abd9875742b0d5eeb0c54ea8f94c0741166

Upstream Comparison:

Branch GIT ID: 2b6af22d5ab7595d8eb09194c29fcabdbe743459
Jenkins Build URL: Acceptance Upstream DXP (master) #3576

ci:test:stable - 27 out of 27 jobs PASSED
27 Successful Jobs:
ci:test:relevant - 189 out of 196 jobs PASSED
189 Successful Jobs:
For more details click here.

This pull contains no unique failures.


Failures in common with acceptance upstream results at 2b6af22:
  1. central-requirements-jdk8/0/0
    1. com.liferay.portal.modules.ModulesStructureTest.testScanBuildScripts
      junit.framework.AssertionFailedError: Version must match the project version (4.0.27) modules/apps/commerce/commerce-shipment-web/package.json
        at com.liferay.portal.modules.ModulesStructureTest._testJSONVersion(ModulesStructureTest.java:1540)
        at com.liferay.portal.modules.ModulesStructureTest.access$500(ModulesStructureTest.java:68)
        at com.liferay.portal.modules.ModulesStructureTest$2.preVisitDirectory(ModulesStructureTest.java:305)
        at com.liferay.portal.modules.ModulesStructureTest$2.preVisitDirectory(ModulesStructureTest.java:168)
        at java.nio.file.Files.walkFileTree(Files.java:2677)
        at java.nio.file.Files.walkFileTree(Files.java:2742)
        at com.liferay.portal.modules.ModulesStructureTest.testScanBuildScripts(ModulesStructureTest.java:166)
      
    2. com.liferay.portal.modules.ModulesStructureTest.testScanGradleFiles
      junit.framework.AssertionFailedError: Redundant dependency detected in modules/apps/portal-search-elasticsearch7/portal-search-elasticsearch7-impl/build.gradle expected:<compileOnly group: "com.liferay", name: "biz.aQute.bnd.annotation", version: "4.2.0.LIFERAY-PATCHED-2"> but was:<testCompile group: "com.liferay", name: "biz.aQute.bnd.annotation", version: "4.2.0.LIFERAY-PATCHED-2">
        at com.liferay.portal.modules.ModulesStructureTest._testGradleFile(ModulesStructureTest.java:1509)
        at com.liferay.portal.modules.ModulesStructureTest.access$800(ModulesStructureTest.java:68)
        at com.liferay.portal.modules.ModulesStructureTest$4.visitFile(ModulesStructureTest.java:402)
        at com.liferay.portal.modules.ModulesStructureTest$4.visitFile(ModulesStructureTest.java:376)
        at java.nio.file.Files.walkFileTree(Files.java:2670)
        at java.nio.file.Files.walkFileTree(Files.java:2742)
        at com.liferay.portal.modules.ModulesStructureTest.testScanGradleFiles(ModulesStructureTest.java:374)
      
  2. modules-integration-mysql57-jdk8/0/12
    1. com.liferay.portal.log.assertor.PortalLogAssertorTest.testScanXMLLog
      junit.framework.AssertionFailedError: 
      null
      com.liferay.portal.kernel.exception.PortalException: Unable to search with filter ((customField/_44822_Zcw9u6Xn eq '0A3RAUD5')) and (classPK eq '20126')
        at com.liferay.segments.internal.odata.search.ODataSearchAdapterImpl.searchCount(ODataSearchAdapterImpl.java:109)
        at com.liferay.segments.internal.odata.retriever.UserODataRetriever.getResultsCount(UserODataRetriever.java:69)
        at com.liferay.segments.internal.provider.BaseSegmentsEntryProvider.isMember(BaseSegmentsEntryProvider.java:293)
        at com.liferay.segments.internal.provider.BaseSegmentsEntryProvider.lambda$getSegmentsEntryIds$2(BaseSegmentsEntryProvider.java:160)
        at java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:174)
        at java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:175)
        at java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1374)
        at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
        at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
        at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:545)
        at java.util.stream.AbstractPipeline.evaluateToArrayNode(AbstractPipeline.java:260)
        at java.util.stream.LongPipeline.toArray(LongPipeline.java:484)
        at com.liferay.segments.internal.provider.BaseSegmentsEntryProvider.getSegmentsEntryIds(BaseSegmentsEntryProvider.java:164)
        at com.liferay.segments.internal.provider.SegmentsEntryProviderRegistryImpl.getS...
  3. ...

✔️ ci:test:sf - 1 out of 1 jobs passed in 3 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 22411abd9875742b0d5eeb0c54ea8f94c0741166

Sender Branch:

Branch Name: LPS-143397_change_tracking
Branch GIT ID: 43647871ec4b7a8d8f34b7ee981bde16416710bc

1 out of 1jobs PASSED
1 Successful Jobs:
For more details click here.

@liferay-continuous-integration
Copy link
Collaborator Author

To conserve resources, the PR Tester does not automatically run for forwarded pull requests.

@brianchandotcom
Copy link
Owner

Merged. Thank you.
View total diff: 8c2467d...6d948b7

@brianchandotcom
Copy link
Owner

@victorg1991 that's a lot of style lint disables?

@victorg1991
Copy link

Hey @brianchandotcom this is required since the stylelint plugin needs some extra configuration. I have opened liferay/liferay-frontend-projects#974 to fix it

@liferay-continuous-integration liferay-continuous-integration deleted the ci-forward-LPS-143397_change_tracking-pr-9002-sender-victorg1991-ts-1658333216793 branch July 28, 2022 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants