1405958 - Update netmonitor doc to reflect current structure

Status: RESOLVED FIXED

(DevTools :: Netmonitor, enhancement)

Description

[[e-mail]]

Current netmonitor doc has some outdate info and we can improve that.

Comment 1

[[e-mail]]

Attachment: Bug 1405958 - Update netmonitor doc to reflect current structure;

Review commit: https://reviewboard.mozilla.org/r/186680/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/186680/

Comment 2

[Ricky Chien [:rickychien] (inactive)]

Comment on attachment 8915477 [details]
Bug 1405958 - Update netmonitor doc to reflect current structure;

https://reviewboard.mozilla.org/r/186680/#review191740

LGTM. r+ and please fix one issue below.

::: devtools/client/netmonitor/README.md:16
(Diff revision 1)
> -Once `node` (`npm` included) is installed, use the following command to install `yarn`.
> +Once `node` (`npm` included) is installed, use the following command to install `yarn`, the pacakage dependency management tool.
>  
>  ```
>  $ npm install -g yarn
>  ```

I'd prefer to remove this installation guide and use official guide instead https://yarnpkg.com/docs/install. There are plenty of ways to install `yarn`, we should let user decide on their own.

Comment 3

[[e-mail]]

Comment on attachment 8915477 [details]
Bug 1405958 - Update netmonitor doc to reflect current structure;

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/186680/diff/1-2/

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5ba1e7396bd8
Update netmonitor doc to reflect current structure;r=rickychien

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/5ba1e7396bd8