Skip to content

docs: standardize npm cli commands and args#3875

Merged
mcollina merged 4 commits intofastify:mainfrom
Fdawgs:docs/tidy
May 3, 2022
Merged

docs: standardize npm cli commands and args#3875
mcollina merged 4 commits intofastify:mainfrom
Fdawgs:docs/tidy

Conversation

@Fdawgs
Copy link
Copy Markdown
Member

@Fdawgs Fdawgs commented May 3, 2022

This PR changes the following:

  • Replaces npm install with npm i alias, as there was a mishmash of both being used throughout docs, so thought it better to be consistent and go with the shorter alias
  • Replaces --save-dev npm arg with -D alias, as there was also a mishmash of both being used throughout docs
  • Removes --save npm arg, which is legacy and hasn't been needed since pre-npm 5.x
  • Replaces legacy --only=production npm arg with --production, as it was deprecated in npm 6.x

Checklist

Copy link
Copy Markdown
Member

@RafaelGSS RafaelGSS left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina merged commit 08cece7 into fastify:main May 3, 2022
@Fdawgs Fdawgs deleted the docs/tidy branch May 3, 2022 14:32
Fdawgs added a commit to fastify/aws-lambda-fastify that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/busboy that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/csrf that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-accepts that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-accepts-serializer that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-caching that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-citgm that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-cli that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-diagnostics-channel that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-env that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-flash that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-jwt that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-kafka that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-leveldb that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-mongodb that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-multipart that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-mysql that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-nextjs that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-oauth2 that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-passport that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-postgres that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-redis that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-soap-client that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-static that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-swagger that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-type-provider-typebox that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-websocket that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/fastify-zipkin that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/forwarded that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/middie that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/point-of-view that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/pre-commit that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/proxy-addr that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/secure-json-parse that referenced this pull request Jul 25, 2022
Fdawgs added a commit to fastify/session that referenced this pull request Jul 25, 2022
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2023

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants