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

๐Ÿ› ๏ธ Tweak internal CI actions, add --no-cache-dir at Dockfile files #52

Merged
merged 3 commits into from
Jul 28, 2023

Conversation

realFranco
Copy link
Contributor

@realFranco realFranco commented Nov 5, 2022

Introduce a --no-cache-dir tag into the pip install command in order to reduce the resulting container size, also at this particular time, the docker containers do not make use of the cache directory on next execution layers, so there is no reason to create it.

See: hadolint/DL3042

@github-actions
Copy link

github-actions bot commented Nov 5, 2022

๐Ÿ“ Docs preview for commit 5116380 at: https://636661d515da5d6ba2e2a75f--asyncer.netlify.app

@github-actions
Copy link

๐Ÿ“ Docs preview for commit 7bfa089 at: https://639cfbd958063717942345d2--asyncer.netlify.app

@tiangolo tiangolo changed the title Add --no-cache-dir arg. at Dockfile files ๐Ÿ› ๏ธ Tweak internal CI actions, add --no-cache-dir at Dockfile files Jul 28, 2023
@tiangolo
Copy link
Member

Makes sense, thanks @realFranco! ๐Ÿš€

@tiangolo tiangolo enabled auto-merge (squash) July 28, 2023 13:35
@github-actions
Copy link

๐Ÿ“ Docs preview for commit 527a15b at: https://64c3c43ea5096d7837f67a46--asyncer.netlify.app

@tiangolo tiangolo merged commit f1d7405 into fastapi:main Jul 28, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants