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-93362 Fix built themes pulling in wrong version of clay-css dependency #70786
Conversation
…dency Instead of relying on the stale snapshots of liferay-frontend-theme-styled and liferay-frontend-theme-unstyled, which pull in an old version of clay-css, always use the in-repo versions directly. Test plan: See that all themes build and deploy, ie. in "modules/apps/", run: for X in admin classic; do (cd frontend-theme/frontend-theme-$X && gradlew clean deploy) done for X in fjord westeros-bank porygon; do (cd frontend-theme-$X/frontend-theme-$X && gradlew clean deploy) done And see these lines in the build output: Files of project ':apps:frontend-theme:frontend-theme-admin' deployed to /Users/greghurrell/code/portal/bundles/deploy Files of project ':apps:frontend-theme:frontend-theme-classic' deployed to /Users/greghurrell/code/portal/bundles/osgi/war Files of project ':apps:frontend-theme-fjord:frontend-theme-fjord' deployed to /Users/greghurrell/code/portal/bundles/deploy Files of project ':apps:frontend-theme-westeros-bank:frontend-theme-westeros-bank' deployed to /Users/greghurrell/code/portal/bundles/deploy Files of project ':apps:frontend-theme-porygon:frontend-theme-porygon' deployed to /Users/greghurrell/code/portal/bundles/deploy<Paste> Look at portal and see header font has reverted to previous, desired size. Note that the yarn.lock no longer pulls in the old clay-css.
Great, thanks @wincent! 👍 |
ci:test-sf |
ci:test-relevant |
ci:stop |
❌ ci:test - 0 out of 1 jobs passed in 53 minutes 45 seconds 873 msClick here for more details.Base Branch:Branch Name: master Copied in Private Modules Branch:Branch Name: master-private 1 Failed Jobs:For more details click here.Failures unique to this pull:For upstream results, click here. |
The pull request tester is still running.Please wait until you get the final report before running 'ci:retest'. See this link to check on the status of your test: However, the pull request was closed.The pull request was closed because the following critical batches had failed: For information as to why we automatically close out certain pull requests see this article. *This pull will no longer automatically close if this comment is available. If you believe this is a mistake please reopen this pull by entering the following command as a comment. ci:reopen Critical Failure Details:test-portal-acceptance-pullrequest-batch(master)/modules-unit-jdk8Job Results:4744 Tests Passed.
cc @
|
Instead of relying on the stale snapshots of liferay-frontend-theme-styled and liferay-frontend-theme-unstyled, which pull in an old version of clay-css, always use the in-repo versions directly.
Test plan:
See that all themes build and deploy, ie. in "modules/apps/", run:
And see these lines in the build output:
Look at portal and see header font has reverted to previous, desired size. Note that the yarn.lock no longer pulls in the old clay-css. Also, create three sites with the three marketplaces themes (porygon, westeros-bank and fjord) and see them all render correctly.
cc: @jbalsas