Skip to content

Commit

Permalink
Remove nvm
Browse files Browse the repository at this point in the history
As part of ITHC report removing nvm.sh installation. For API nvm is used to install `squawk-cli` so now installing `squawk-cli` via pip and removing nvm.
  • Loading branch information
spatel033 committed Jan 9, 2025
1 parent 964ad03 commit 538e1f5
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 13 deletions.
13 changes: 0 additions & 13 deletions docker/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -162,16 +162,3 @@ RUN \
RUN \
service postgresql start \
&& su - postgres -c "psql -c \"alter user postgres with password 'postgres';\""


ENV NODE_VERSION=20.10.0
ENV NVM_VERSION=0.39.7

RUN \
echo "Installing NVM & Node" \
&& curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v${NVM_VERSION}/install.sh | bash \
&& source ~/.nvm/nvm.sh \
&& nvm install $NODE_VERSION \
&& npm install -g [email protected]

ENV PATH=/root/.nvm/versions/node/v${NODE_VERSION}/bin:$PATH
1 change: 1 addition & 0 deletions requirements_for_test.in
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,4 @@ flaky==3.8.0
moto==5.0.11
pytest-httpserver==1.0.8
trustme==0.9.0
squawk-cli==1.4.0
2 changes: 2 additions & 0 deletions requirements_for_test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -366,6 +366,8 @@ sqlalchemy==1.4.41
# alembic
# flask-sqlalchemy
# marshmallow-sqlalchemy
squawk-cli==1.4.0
# via -r requirements_for_test.in
statsd==4.0.1
# via
# -r requirements.txt
Expand Down

0 comments on commit 538e1f5

Please sign in to comment.