Allow production builds for UI packages #3269
Open
+42
−30
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this pull request?
Targets in the Makefile that touch the UI building process now have
prod
anddev
versions. The UI packages offer both dev and prod scripts but currently, we are using only the dev version, which is useful only for development, not for the production environment. Thedev
version is larger since source maps and unoptimized files for the UI packages are included. Now, theprod
version does not include source maps, and the source is optimized for production.Cherry-picked from ODH Elyra:
How was this pull request tested?
Build Elyra with both
make clean install-all-dev
andmake clean install-all-prod
, and check the resultingwhl
file size and contents.Also the CI on my fork: https://github.com/caponetto/opendatahub-io-elyra/actions/runs/12582298698
Developer's Certificate of Origin 1.1