Skip to content

Update suggested npm production Dockerfile#2458

Merged
pwizla merged 1 commit into
strapi:mainfrom
ClaXXX:patch-1
Apr 11, 2025
Merged

Update suggested npm production Dockerfile#2458
pwizla merged 1 commit into
strapi:mainfrom
ClaXXX:patch-1

Conversation

@ClaXXX

@ClaXXX ClaXXX commented Apr 10, 2025

Copy link
Copy Markdown
Contributor

What does it do?

Updates in documentation the npm production Dockerfile -> https://docs.strapi.io/cms/installation/docker#production-dockerfile.

  • Fixes docker warning about the as being in a different case
  • Fixes docker warning about usage of deprecated ENV call
  • Bumps node image number to the latest supported one

Why is it needed?

To keep an up-to-date production Dockerfile and let anyone copy-paste a valid code.

Related issue(s)/PR(s)

Didn't found any

+ Fixes docker warning about the as being in a different case
+ Fixes docker warning about usage of deprecated ENV call
+ Bumps node image number to the latest supported one
@ClaXXX ClaXXX requested a review from meganelacheny as a code owner April 10, 2025 14:00
@vercel

vercel Bot commented Apr 10, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 10, 2025 2:08pm

@strapi-cla

strapi-cla commented Apr 10, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@pwizla pwizla left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you, @ClaXXX!
The PR is now approved, merged, should be live on docs.strapi.io in a few minutes, and will be mentioned in the next weekly snapshot release.

@pwizla pwizla self-assigned this Apr 11, 2025
@pwizla pwizla added this to the 6.2.0 milestone Apr 11, 2025
@pwizla pwizla merged commit 5ef96c2 into strapi:main Apr 11, 2025
@pwizla pwizla changed the title Documentation: Update suggested npm production Dockerfile Update suggested npm production Dockerfile Apr 11, 2025
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.

3 participants