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.

Sign up for GitHub

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

Jump to bottom

npm install and start fail with the local stack #73

Closed
velramiir opened this issue Sep 28, 2023 · 0 comments · Fixed by #74
Closed

npm install and start fail with the local stack #73

velramiir opened this issue Sep 28, 2023 · 0 comments · Fixed by #74
Assignees
@velramiir

Comments

@velramiir
Copy link
Contributor

🔥 Bug Description

When using the local dotbase stack, both npm install and npm start fail with the error that the npm cache contains root-owned files. The reason is that the home dir in the development container is the root dir / and npm tries to create the cache at ~/.npm by default.

🔍 Steps to Reproduce the Bug

  1. Launch stack from icd-10-api repo
  2. Run either command in the docker container of the icd-10-api

🧯 References and Tools
@velramiir velramiir self-assigned this Sep 28, 2023
@velramiir velramiir mentioned this issue Sep 28, 2023
Merged
4 tasks
dotbase-bot pushed a commit that referenced this issue Sep 29, 2023
@velramiir
fix: home needs to be valid npm cache location (#74)
da453b1 
# Ticket 🎫

This closes #73.

# Description 📖
Changes the home dir in the dev overlay to `/tmp` just like for the
medical-dashboard.

# How to Test? 🧪

See bug reproduction steps.

# References (optional) 🔗

See issue references.

# Checklist ✅
- [x] My changes generate no new warnings
- [x] Made corresponding changes to the documentation if applicable
- [x] Commented my code, particularly in hard-to-understand areas if
applicable
- [x] Performed a self-review of my own changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@velramiir velramiir

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: home needs to be valid npm cache location dot-base/icd-10-api
2 participants
@velramiir @andreakreichgauer