Closed bai1024 closed 4 months ago
I'm still needing to find time to review this, but remembered we will also need to ensure this still works with a docker container. In that case, we likely want a muti build image that builds the javascript in it's own image and then copies it over to the release image.
I'm still needing to find time to review this, but remembered we will also need to ensure this still works with a docker container. In that case, we likely want a muti build image that builds the javascript in it's own image and then copies it over to the release image.
@bai1024 , I have created a pull request for the necessary changes to the Makefile
and the Dockerfile
. After you merge my commits into your branch, you can squash them all together and update this PR.
Also, I noticed you tried to remove the generated files from this PR with this commit: f70e159. However, I think during the rebase you did after that, the commit was removed and the generated files are back on this PR. Please check it out.
Internally we discussed handling some of this a different way, so will close this PR for now.
This PR shows the build pipeline for Primevue Before try to use the new components, we need to import them first in
promgen-gen/main.js
Then runmake primevue
command. After the new file is generated understatic/promgen
. You can start to use the new components!