admin/twenty
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

feat: replace ts-node with tsx and pre-install it globaly in docker (#7027)

Browse Source 
Implement https://github.com/twentyhq/twenty/issues/5976 suggestion.
It leads to faster boot time since we don't have to install
dependencies.

It should also solve https://github.com/twentyhq/twenty/issues/4772 as
we don't need to run ressource-heavy `npm install` (the mentioned
migration step does not seem to exist anymore anyway.
This commit is contained in:
Quentin G.
2024-09-14 11:05:01 +02:00
committed by GitHub
parent b0bc839a49
commit d1871a4a64
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/twenty-docker/twenty/Dockerfile
View File

@ -50,6 +50,8 @@ FROM node:18.17.1-alpine as twenty
# Used to run healthcheck in docker # Used to run healthcheck in docker
RUN apk add --no-cache curl jq RUN apk add --no-cache curl jq
RUN npm install -g tsx
COPY ./packages/twenty-docker/twenty/entrypoint.sh /app/entrypoint.sh COPY ./packages/twenty-docker/twenty/entrypoint.sh /app/entrypoint.sh
RUN chmod +x /app/entrypoint.sh RUN chmod +x /app/entrypoint.sh

2
packages/twenty-docker/twenty/entrypoint.sh
View File

@ -5,7 +5,7 @@ if [ "${ENABLE_DB_MIGRATIONS}" = "true" ] && [ ! -f /app/docker-data/db_status ]
echo "Running database setup and migrations..." echo "Running database setup and migrations..."
# Run setup and migration scripts # Run setup and migration scripts
NODE_OPTIONS="--max-old-space-size=1500" npx ts-node ./scripts/setup-db.ts NODE_OPTIONS="--max-old-space-size=1500" tsx ./scripts/setup-db.ts
yarn database:migrate:prod yarn database:migrate:prod
# Mark initialization as done # Mark initialization as done