Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update rpc_console docs for building and running #16801

Merged
merged 2 commits into from
Apr 5, 2022

Conversation

cpagravel
Copy link
Contributor

@cpagravel cpagravel commented Mar 29, 2022

Problem

What is being fixed? Examples:

  • Clarify location of which files to build and to install.
  • Add note that python -m chip_rpc.console can be used instead of chip-console to imply what the entry point is.

Change overview

Clarify process for building and installing rpc_console by specifying folder locations for build actions.

Testing

How was this tested? (at least one bullet point required)

  • I re-ran this process myself to confirm that the updated docs are correct.

@CLAassistant
Copy link

CLAassistant commented Mar 29, 2022

CLA assistant check
All committers have signed the CLA.

@woody-apple woody-apple merged commit 913aca0 into project-chip:master Apr 5, 2022
chencheung pushed a commit to chencheung/connectedhomeip that referenced this pull request Apr 6, 2022
* Update rpc_console docs for building and running

* Restyled by prettier-markdown

Co-authored-by: Restyled.io <[email protected]>
chencheung pushed a commit to chencheung/connectedhomeip that referenced this pull request Apr 6, 2022
* Update rpc_console docs for building and running

* Restyled by prettier-markdown

Co-authored-by: Restyled.io <[email protected]>
andrei-menzopol pushed a commit to andrei-menzopol/connectedhomeip that referenced this pull request Apr 14, 2022
* Update rpc_console docs for building and running

* Restyled by prettier-markdown

Co-authored-by: Restyled.io <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants