Yarn3 publish command

[hindicator]

Code of Conduct

• I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?

content\actions\publishing-packages\publishing-nodejs-packages.md

What part(s) of the article would you like to see updated?

// For Yarn3 use -
// yarn npm publish

• run: yarn publish

lines 180 - 183

Additional information

I should fork the repo only for such a small suggestion instead of branch push and PR ?
I believe the comment fits, your choice.
Have a great day =).

[welcome]

Thanks for opening this issue. A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

[cmwilson21]

@hindicator Thanks for opening an issue! We have instructions for Yarn included. Would you mind clarifying the change you are suggesting?

[hindicator]

@cmwilson21
Yarn 3 command for publish is
yarn npm publish
Unlike the written command yarn publish
So my suggestion is to add a comment before, mentioning this.

[cmwilson21]

Gotcha! Thank you for clarifying! I see the difference now 😄

I'll get this triaged for review! ⚡

[skedwards88]

Thanks for opening this issue! It looks like Yarn 1 is not being maintained, so I would be ok with updating this example to use yarn npm publish (which was introduced in Yarn 2), and adding a comment to use yarn publish for Yarn 1. (Basically just the inverse of what you recommended.) e.g. something like

      - run: yarn
      - run: yarn npm publish // for Yarn version 1, use `yarn publish` instead

You or anyone else is welcome to open a PR with this change.