💚 fixing docker environment variables, and rebuild conditions (#674)
`RUN export env=val` in docker doesn't persist the environment variable for following `RUN` commands. Also the `publish_feature.yml` didn't trigger when modifying the relevant `Dockerfile`s
This commit is contained in:
@@ -21,14 +21,19 @@ jobs:
|
|||||||
filters: |
|
filters: |
|
||||||
backend:
|
backend:
|
||||||
- "tdrive/backend/node/**"
|
- "tdrive/backend/node/**"
|
||||||
|
- "tdrive/docker/tdrive-node/Dockerfile"
|
||||||
frontend:
|
frontend:
|
||||||
- "tdrive/frontend/**"
|
- "tdrive/frontend/**"
|
||||||
|
- "tdrive/docker/tdrive-frontend/**"
|
||||||
onlyoffice-connector:
|
onlyoffice-connector:
|
||||||
- "tdrive/connectors/onlyoffice-connector/**"
|
- "tdrive/connectors/onlyoffice-connector/**"
|
||||||
|
- "tdrive/docker/onlyoffice-connector/Dockerfile"
|
||||||
ldap-sync:
|
ldap-sync:
|
||||||
- "tdrive/backend/utils/ldap-sync/**"
|
- "tdrive/backend/utils/ldap-sync/**"
|
||||||
|
- "tdrive/docker/tdrive-ldap-sync/Dockerfile"
|
||||||
nextcloud-migration:
|
nextcloud-migration:
|
||||||
- "tdrive/backend/utils/nextcloud-migration/**"
|
- "tdrive/backend/utils/nextcloud-migration/**"
|
||||||
|
- "tdrive/docker/tdrive-nextcloud-migration/Dockerfile"
|
||||||
|
|
||||||
publish-feature:
|
publish-feature:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|||||||
@@ -23,11 +23,11 @@ FROM node-base AS installed-libs
|
|||||||
|
|
||||||
COPY backend/node/ .
|
COPY backend/node/ .
|
||||||
#Install dev dependancies for build
|
#Install dev dependancies for build
|
||||||
RUN export NODE_ENV=development
|
ENV NODE_ENV=development
|
||||||
RUN npm ci
|
RUN npm ci
|
||||||
|
|
||||||
#Build in production mode
|
#Build in production mode
|
||||||
RUN export NODE_ENV=production
|
ENV NODE_ENV=production
|
||||||
RUN npm run build
|
RUN npm run build
|
||||||
RUN rm -rf node_modules
|
RUN rm -rf node_modules
|
||||||
#Install prod dependancies after build
|
#Install prod dependancies after build
|
||||||
|
|||||||
Reference in New Issue
Block a user