-
Notifications
You must be signed in to change notification settings - Fork 43
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
DOCS-1116: Update viam CLI brew install #1820
DOCS-1116: Update viam CLI brew install #1820
Conversation
Overall readability score: 54.7 (🟢 +0)
View detailed metrics🟢 - Shows an increase in readability
Averages:
View metric targets
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
{{% alert title="Tip" color="tip" %}} | ||
You can use the `uname -m` command to determine your system architecture. | ||
{{% /alert %}} | ||
If you are on Linux, you can use the `uname -m` command to determine your system architecture. | ||
|
||
{{< tabs >}} | ||
{{% tab name="Linux aarch64" %}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
out of scope for PR, but should macos be first here? (on the assumption it's the most common case)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, works for me!
docs/manage/CLI.md
Outdated
@@ -95,7 +82,8 @@ echo 'export PATH="$HOME/go/bin:$PATH"' >> ~/.bashrc | |||
{{% /tab %}} | |||
{{< /tabs >}} | |||
|
|||
To later update the Viam CLI tool, you can use the steps above to reinstall the latest version. | |||
To later update the Viam CLI tool on Linux, use the steps above to reinstall the latest version. | |||
to later update the Viam CLI tool on macOS, run `brew upgrade viamrobotics/brews/viam`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if they went through brew tap
above, can also do brew upgrade viam
, but this works too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point!
You can view a rendered version of the docs from this PR at https://docs-test.viam.dev/bbe4be370ef123ac8412bf2b52f052832dae6c9f/public |
Removed both macOS arch direct download steps in favor of new
brew
install.uname -m
tip, since it now only applies to a subset of installation paths)Staging