Skip to content
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

Add Dockerfile #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add Dockerfile #1

wants to merge 3 commits into from

Conversation

pcvolkmer
Copy link

@pcvolkmer pcvolkmer commented Sep 27, 2022

This PR adds a Dockerfile to build a Docker image for bwHC Frontend and removes directory node_modules from sources.

The node_modules directory was removed to reduce amount of files in Git repository and to reduce the size of the
Docker build context. It needs to be (re)created for local and/or CI/CD builds using npm install command. The install
script in install.sh also uses this to install node dependencies.

The Dockerfile is able to modify package.json and nuxt.config.js to customize specific values for the build
such as protocol, hostname or port of backend installation.

@hannes-voss
Copy link

@aydincan REMINDER: This PR is very useful even for other institutes at other locations!

pcvolkmer and others added 3 commits July 10, 2023 16:21
Use `npm install` to install required node modules as defined in
package.json and package-lock.json.
This adds a Dockerfile to build bwHC Frontend from current sources.

By passing arguments to the docker build command, the build is able to
modify values in file `nuxt.config.js`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants