-
Notifications
You must be signed in to change notification settings - Fork 0
V9.0.3/package maintenance #6
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
Changes from all commits
1979e13
4cfcc7e
2d75005
803d47d
1cd6f3a
3f0793c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,14 +1,14 @@ | ||||||||||||||
| FROM nginx:1.27.3-alpine AS base | ||||||||||||||
| FROM --platform=$BUILDPLATFORM nginx:1.27.5-alpine AS base | ||||||||||||||
| RUN rm -rf /usr/share/nginx/html/* | ||||||||||||||
|
|
||||||||||||||
| FROM codebeltnet/docfx:2.77.0 AS build | ||||||||||||||
| FROM --platform=$BUILDPLATFORM codebeltnet/docfx:2.78.3 AS build | ||||||||||||||
|
|
||||||||||||||
| ADD [".", "docfx"] | ||||||||||||||
|
|
||||||||||||||
| RUN cd docfx; \ | ||||||||||||||
| docfx build | ||||||||||||||
|
|
||||||||||||||
| FROM base AS final | ||||||||||||||
| FROM nginx:1.27.5-alpine AS final | ||||||||||||||
| WORKDIR /usr/share/nginx/html | ||||||||||||||
| COPY --from=build /build/docfx/wwwroot /usr/share/nginx/html | ||||||||||||||
|
Comment on lines
+11
to
13
|
||||||||||||||
| FROM nginx:1.27.5-alpine AS final | |
| WORKDIR /usr/share/nginx/html | |
| COPY --from=build /build/docfx/wwwroot /usr/share/nginx/html | |
| FROM base AS final | |
| WORKDIR /usr/share/nginx/html | |
| COPY --from=build /build/docfx/wwwroot /usr/share/nginx/html |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,10 @@ | ||
| Version 9.0.2 | ||
| Version 9.0.3 | ||
| Availability: .NET 9, .NET 8 and .NET Standard 2.0 | ||
|
|
||
| # ALM | ||
| - CHANGED Dependencies have been upgraded to the latest compatible versions for all supported target frameworks (TFMs) | ||
|
|
||
| Version 9.0.2 | ||
|
||
| Availability: .NET 9, .NET 8 and .NET Standard 2.0 | ||
|
|
||
| # ALM | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Unused cleanup stage – refactor or remove
The base stage cleans
/usr/share/nginx/htmlbut is never used in the final image. To ensure the cleanup step applies, either remove the base stage entirely or derive the final stage from it.Example change:
This preserves the cleanup before copying the built content.
🤖 Prompt for AI Agents