Merge pull request #845 from OpenChemistry/sentry-take-2

Remove sentry build args
OpenChemistry · Jul 11, 2023 · e286fd4 · e286fd4
2 parents 99582de + 4ce5b1b
commit e286fd4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/frontend/Dockerfile b/frontend/Dockerfile
@@ -1,8 +1,7 @@
 FROM node:18.6.0 as build
 COPY ./distiller /distiller
 ARG REACT_APP_API_URL
-ARG REACT_APP_SENTRY_DSN_URL
-RUN cd /distiller && yarn && REACT_APP_API_URL=$REACT_APP_API_URL REACT_APP_SENTRY_DSN_URL=$REACT_APP_SENTRY_DSN_URL yarn build
+RUN cd /distiller && yarn && REACT_APP_API_URL=$REACT_APP_API_URL yarn build
 
 FROM nginx
 COPY --from=build /distiller/build/ /usr/share/nginx/html

diff --git a/frontend/scripts/docker_build.sh b/frontend/scripts/docker_build.sh
@@ -2,4 +2,4 @@
 
 TAG=`git log -1 --pretty=%h`
 
-docker build --build-arg REACT_APP_SENTRY_DSN_URL=$REACT_APP_SENTRY_DSN_URL -t openchemistry/distiller-client:$TAG -f ../Dockerfile ../ $@
+docker build -t openchemistry/distiller-client:$TAG -f ../Dockerfile ../ $@