Skip to content

Upgrade base image to use PowerShell 7.4.11#1511

Merged
VeryEarly merged 3 commits into
mainfrom
yabo/upgrade-baseimage
Sep 3, 2025
Merged

Upgrade base image to use PowerShell 7.4.11#1511
VeryEarly merged 3 commits into
mainfrom
yabo/upgrade-baseimage

Conversation

@VeryEarly

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread tools/docker/Dockerfile
@@ -1,4 +1,4 @@
FROM mcr.microsoft.com/powershell:lts-7.2-ubuntu-22.04
FROM mcr.microsoft.com/dotnet/sdk:8.0.413-noble

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noble is Ubuntu 23, which is not a LTS version. Do we have image of Ubuntu 24.

Comment thread tools/docker/Dockerfile Outdated
&& apt-get -o Acquire::Check-Valid-Until=false install -y \
&& apt-get clean \
&& rm -rf /var/lib/apt/lists/* \
&& apt-get update \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is duplicated with line#4.

Comment thread tools/docker/Dockerfile Outdated
&& apt-get clean \
&& rm -rf /var/lib/apt/lists/* \
&& apt-get update \
&& apt-get install -y locales \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we move this line up to be with the other install instructions so that apt-get clean could clean the package cache.

@VeryEarly
VeryEarly merged commit d47114e into main Sep 3, 2025
104 checks passed
@VeryEarly
VeryEarly deleted the yabo/upgrade-baseimage branch September 3, 2025 08:15
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.

2 participants