Skip to content

Improve setup instructions#6156

Merged
samcoe merged 3 commits intocli:trunkfrom
edhgoose:improve-setup-instructions
Sep 2, 2022
Merged

Improve setup instructions#6156
samcoe merged 3 commits intocli:trunkfrom
edhgoose:improve-setup-instructions

Conversation

@edhgoose
Copy link
Contributor

I visited Google to find the cli install instructions, and ended up following these steps:

  1. Link ends up on this repository (github.com/cli/cli)
  2. Scroll down to usage and installation instructions
  3. Click link to "see the manual" which visits https://cli.github.com/manual/
  4. Click on the link to see installation instructions, which ends up back at the readme here.

This little change makes it clearer the installation instructions are below. Wording improvement welcome!

@edhgoose edhgoose requested a review from a team as a code owner August 29, 2022 16:19
@edhgoose edhgoose requested review from samcoe and removed request for a team August 29, 2022 16:19
@cliAutomation cliAutomation added the external pull request originating outside of the CLI core team label Aug 29, 2022
@cliAutomation
Copy link
Collaborator

Hi! Thanks for the pull request. Please ensure that this change is linked to an issue by mentioning an issue number in the description of the pull request. If this pull request would close the issue, please put the word 'Fixes' before the issue number somewhere in the pull request body. If this is a tiny change like fixing a typo, feel free to ignore this message.

@samcoe samcoe self-assigned this Aug 30, 2022
Copy link
Contributor

@samcoe samcoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for helping improve our docs. I proposed a slight wording change, let me know what you think.

@edhgoose
Copy link
Contributor Author

edhgoose commented Sep 2, 2022

Sounds good @samcoe - I've made that change.

Copy link
Contributor

@samcoe samcoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for making the requested changes!

@samcoe samcoe enabled auto-merge (squash) September 2, 2022 09:33
@samcoe samcoe merged commit 5f52a1d into cli:trunk Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external pull request originating outside of the CLI core team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants