Skip to content

Commit

Permalink
Add comment explaining reasoning for calling uv sync twice.
Browse files Browse the repository at this point in the history
I was rather confused why there were 2 `uv sync` calls in the file and found out more from `uv help sync`. Added a comment to make it clearer.
  • Loading branch information
valankar authored Sep 10, 2024
1 parent dee88a8 commit ca57929
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ RUN --mount=type=cache,target=/root/.cache/uv \
uv sync --frozen --no-install-project --no-dev

# Then, add the rest of the project source code and install it
# Installing separately from its dependencies allows optimal layer caching
ADD . /app
RUN --mount=type=cache,target=/root/.cache/uv \
uv sync --frozen --no-dev
Expand Down

0 comments on commit ca57929

Please sign in to comment.