Conversation
…add-nodejs-support
neunhoef
left a comment
neunhoef
left a comment
There was a problem hiding this comment.
I have run the service maker on the included shoppinglist-express project and produced a project.tar.gz with it. Then I looked into this and found that the node_modules folder seems to contain all dependencies and not only those which are needed but are not contained in the base image.
I do not understand why this has happened?
It seems the mechanism to only install the additionally needed packages is not working.
Can you please investigate this?
| @@ -0,0 +1,94 @@ | |||
| #!/usr/bin/env node | |||
There was a problem hiding this comment.
Do we really have to do this "manually"? I had hoped that this rather complicated and potentially error-prone analysis would be done by some standard js packaging tool like npm (or whatever).
For the python case we could make use of uv for this.
neunhoef
left a comment
neunhoef
left a comment
There was a problem hiding this comment.
LGTM. Let's leave this for now as it is. We "manually" compute the dependency list. I had hoped that some tool like npm can do this for us, but if not, then we must do it ourselves.
2 participants
No description provided.