pre-commit.Dockerfile: add all files in a single COPY #5562

Merged
tobiasd merged 1 commits from caching into master 2025-12-16 18:38:17 +00:00
Contributor

also removed comment as it is no longer relevant

Signed-off-by: Filip Kobierski fkobi@fsfe.org

I recommend side-by-side diff for reviewing the changes

also removed comment as it is no longer relevant Signed-off-by: Filip Kobierski <fkobi@fsfe.org> I recommend side-by-side diff for reviewing the changes
fkobi added the enhancement label 2025-12-11 09:13:01 +00:00
fkobi requested review from delliott 2025-12-11 09:13:03 +00:00
fkobi force-pushed caching from 7f881f310b to b38ad2ea9b 2025-12-11 09:13:26 +00:00 Compare
delliott approved these changes 2025-12-11 12:15:37 +00:00
delliott left a comment
Member

This is kind of worse for docker caching, as now changes to entry point invalidate the UV install, which is unnecessary and logically incorrect.

But I suppose it uses less layers.

Go ahead, I suppose.

This is kind of worse for docker caching, as now changes to entry point invalidate the UV install, which is unnecessary and logically incorrect. But I suppose it uses less layers. Go ahead, I suppose.
fkobi force-pushed caching from b38ad2ea9b to 1cf09f0c11 2025-12-15 09:41:29 +00:00 Compare
tobiasd merged commit 2a96a13fb8 into master 2025-12-16 18:38:17 +00:00
tobiasd deleted branch caching 2025-12-16 18:38:18 +00:00
Sign in to join this conversation.