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-107202 Update liferay-npm-scripts to v19.0.2 #83439
LPS-107202 Update liferay-npm-scripts to v19.0.2 #83439
Conversation
This file contains nothing not already present in the canonical config file higher up in the repo, and it is going to cause trouble in the next commit anyway (because it adds a lint forcing the use of ".prettierrc.js" instead of ".prettierrc"), so let's just delete it.
The new version switches our Babel default config over to assume React, enabling us to eliminate a bunch of duplicative config. The new version of liferay-npm-scripts also includes lints to make sure that: - The default presets are not redundantly specified in config files. - Config files use JS-based formats that we can easily lint (eg. ".babelrc.js", ".eslintrc.js" etc) instead of others (eg. ".eslintrc" etc). There are two projects where we have to suppress the React preset because their use of incremental-dom is incompatible. Those are dynamic-data-mapping-form-web and dynamic-data-mapping-form-builder. Once we migrate those projects off of incremental-dom, we can remove those config files entirely. Note that both of these have been ported from ".babelrc" (JSON) to ".babelrc.js" so that the aforementioned lints can apply; as such, we get formatting for free too.
Addresses JS test failure mentioned here: brianchandotcom#83438 (comment) Actual fix was: liferay/liferay-npm-tools#343
CI is automatically triggering "ci:test:sf" and "ci:test:relevant" for this pull to run Source Formatter and relevant tests. Comment "ci:test" to run the full PR Tester for this pull. |
In my local testing, I am seeing a JS test failure in
That one is already being tracked in LPS-107247. |
✔️ ci:test:sf - 1 out of 1 jobs passed in 3 minutes 44 seconds 128 msClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: wincent/LPS-107202/liferay-npm-scripts-v19 1 Successful Jobs:For more details click here. |
726fc07
to
ad09799
Compare
Merged. Thx. |
Replacement for #83438 (fixes tests).
TL;DR
From commit 36cf902:
Originally tested here: wincent#120