docs icon indicating copy to clipboard operation
docs copied to clipboard

docker.mdx: use `--omit=dev` for npm over deprecated `--production`

Open corneliusroemer opened this issue 9 months ago • 3 comments

Description (required)

This fixes npm warnings that I get using the current suggested Dockerfile, see warnings:

[linux/arm64 build-deps 1/1] 
RUN npm install --production=false
npm WARN config production Use `--omit=dev` instead.
RUN npm install --production
npm WARN config production Use `--omit=dev` instead.

Related issues & labels (optional)

  • Closes #
  • Suggested label:

corneliusroemer avatar May 08 '24 18:05 corneliusroemer

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

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 14, 2024 0:55am
docs-i18n 🔄 Building (Inspect) Visit Preview 💬 Add feedback May 14, 2024 0:55am

vercel[bot] avatar May 08 '24 18:05 vercel[bot]

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉

Here’s what will happen next:

  1. Our GitHub bots will run to check your changes. If they spot any broken links you will see some error messages on this PR. Don’t hesitate to ask any questions if you’re not sure what these mean!

  2. In a few minutes, you’ll be able to see a preview of your changes on Vercel 🥳

  3. One or more of our maintainers will take a look and may ask you to make changes. We try to be responsive, but don’t worry if this takes a few days.

astrobot-houston avatar May 08 '24 18:05 astrobot-houston

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale File Note
en recipes/docker.mdx Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

astrobot-houston avatar May 08 '24 18:05 astrobot-houston

Thank you everyone for discussing this so thoughtfully and thoroughly!

(And reminder, at this point it's only the submitted change that we're considering. If there's other text on the page someone feels should be changed, that's out of the scope of this proposed change and a separate PR can be made.)

sarah11918 avatar May 14 '24 12:05 sarah11918