Update the documentation for the ls command for the rename to list. #277
Conversation
|
@Daniel15 Fixed that up. Thanks for letting me know. |
There was a problem hiding this comment.
Thanks @danielmcgraw. I accidentally created a PR for the same thing and just commented on some changes my PR had that yours doesn't.
@Daniel15 anything stopping this from being merged?
| tags: ["cli-ls"] | ||
| - id: docs_cli_list | ||
| path: /docs/cli/list | ||
| tags: ["cli-list"] |
There was a problem hiding this comment.
This should reordered to be above login, logout in order to keep the commands in alpha order
| @@ -125,7 +125,7 @@ docs_cli_licenses: yarn licenses | |||
| docs_cli_link: yarn link | |||
| docs_cli_login: yarn login | |||
| docs_cli_logout: yarn logout | |||
| docs_cli_ls: yarn ls | |||
| docs_cli_list: yarn list | |||
There was a problem hiding this comment.
This should be moved above login / logout to retain alpha order.
|
I also encountered mismatch between docs and |
| @@ -10,7 +10,7 @@ layout: guide | |||
|
|
|||
| ##### `yarn global` <a class="toc" id="toc-yarn-global" href="#toc-yarn-global"></a> | |||
|
|
|||
| `yarn global` is a prefix used for a number of commands like `add`, `bin`, `ls` and `remove`. They behave identically to their normal versions except that they use a global directory to store packages. The `global` command makes binaries available to use on your operating system. | |||
| `yarn global` is a prefix used for a number of commands like `add`, `bin`, `list` and `remove`. They behave identically to their normal versions except that they use a global directory to store packages. The `global` command makes binaries available to use on your operating system. | |||
There was a problem hiding this comment.
yarn global list is not valid yet. yarn global ls command remained. v0.18.1
I added an issue to yarn yarn#2354
|
If changes haven't gone live then the docs shouldn't be modified ahead of it. Right now the docs is in need of some adjustment to make it look right: https://yarnpkg.com/en/docs/cli/ls |
- yarnpkg#356 did not create redirect - yarnpkg#334 duplicated some work already done in yarnpkg#333 also did not update all necessary files. - yarnpkg#277 had merge conflicts that went unaddressed. Once this is merged, all of yarnpkg#277, yarnpkg#334 and yarnpkg#356 can be closed.
- yarnpkg#356 did not create redirect - yarnpkg#334 duplicated some work already done in yarnpkg#333 also did not update all necessary files. - yarnpkg#277 had merge conflicts that went unaddressed. Once this is merged, all of yarnpkg#277, yarnpkg#334 and yarnpkg#356 can be closed.
|
Thanks for helping. If I see this correctly, this has been fixed by #377 |
Update to the documentation if/when yarnpkg/yarn#1940 is pulled in to yarn.