Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Warnings in Docker Compose #3120

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ParagEkbote
Copy link

@ParagEkbote ParagEkbote commented Dec 13, 2024

I've updated the Makefiles for docker compose to fix the warnings described in #2289

I also had a query: What environment variable references do I have to set for CACHED_ASSETS_BASE_URL, API_HF_JWT_ADDITIONAL_PUBLIC_KEYS and ASSETS_BASE_URL since there were several different ones used. Please let me know and I will make the necessary changes.

Fixes #2289

cc: @severo

@ParagEkbote
Copy link
Author

Could you please review the changes?

cc: @severo

@severo severo requested a review from lhoestq December 19, 2024 21:29
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.

Remove warnings in docker compose
1 participant