node.js sample from node devfile contains node_modules in the source code

Describe the bug

node.js sample from node devfile(https://github.com/eclipse/che-devfile-registry/blob/master/devfiles/nodejs/devfile.yaml#L10) contains node_modules folder in the source code It’s looks strange . In most case node developers exclude such folder from source code and use package.lock.json or yarn.lock to freeze transitive dependencies, or to use transitive dependencies without security vulnerabilities.

Che version

• latest
• nightly
• other: please specify

Steps to reproduce

1. Create new workspace from node devfile “NodeJS Express Web Application”
2. Open source code project nodejs-web-app

Expected behavior

Sample should has node_modules excluded in the .gitignore file and we should have a command in the devfile to get node dependencies (‘npm install’) or ‘npm install’ should be embedded to the application start devfile command.

Runtime

• kubernetes (include output of kubectl version)
• Openshift (include output of oc version)
• minikube (include output of minikube version and kubectl version)
• minishift (include output of minishift version and oc version)
• docker-desktop + K8S (include output of docker version and kubectl version)
• other: (please specify)

Screenshots

Installation method

• chectl
• che-operator
• minishift-addon
• I don’t know

Environment

• my computer
  • Windows
  • Linux
  • macOS
• Cloud
  • Amazon
  • Azure
  • GCE
  • other (please specify)
• other: please specify

Additional context

[slemeur]

It is useful, but I don’t think it is high priority, but as it’s something fairly quick to handle we could take it.

[amisevsk]

@slemeur I believe we could remove node_modules since we own the repo, I was planning on opening a PR for that and other changes to sample projects at some point (e.g. the web-java-spring sample project readme is out of date.

Do you think cleaning up sample projects is something useful for 7.0.0?