Skip to content

Conversation

@Carbrex
Copy link
Member

@Carbrex Carbrex commented Apr 18, 2024

Addind pnpm-store to volumes doesn't download the dependencies again and again on running ./lila-docker ui and other command which include pnpm install.
Before:
image
Downloaded packages every time on pnpm install
As far as I understand this is because docker compose run --rm ui /lila/ui/build --update --clean-build --debug $@ makes a new container every time it is run.
After:
image
Just takes the package from pnpm-store directory and doesn't download them again.

@Carbrex
Copy link
Member Author

Carbrex commented Apr 18, 2024

Also one more thing running ../../lila-docker db from other directory say /repos/lila gives error as settings.env not found. Not sure why the cd "$(dirname "$0")" was removed from lila-docker script.

@fitztrev
Copy link
Member

Not sure why the cd "$(dirname "$0")" was removed from lila-docker script.

Unintentional when I changed it to work with multiple-word commands in #61

@fitztrev fitztrev merged commit e352c49 into lichess-org:main Apr 20, 2024
@fitztrev
Copy link
Member

Thanks. Wonder why .pnpm-store is already in gitignore.

@Carbrex Carbrex deleted the pnpm-store-vol branch July 4, 2024 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants