Add installation script#772
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds a shell-based installation script for GitHub Copilot CLI to support quick installation on macOS and Linux systems via curl or wget, addressing issue #771.
- Adds install.sh script that detects platform/architecture and downloads the appropriate binary
- Updates README.md with new installation methods organized by platform
- Provides alternative installation path for users who prefer not to use npm
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 8 comments.
| File | Description |
|---|---|
| install.sh | New installation script that handles platform detection, binary download/extraction, and installation to user's local bin directory |
| README.md | Reorganizes installation section with platform-specific subsections and adds curl/wget installation methods |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 8 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
andyfeller
left a comment
andyfeller
left a comment
There was a problem hiding this comment.
Our users will be spoiled for choice!
Just a handful of thoughts with my main concern being the readme changes.
| npm install -g @github/copilot | ||
| ``` | ||
|
|
||
| #### macOS and Linux |
There was a problem hiding this comment.
suggest: as a user, I would find it easier if the installation sections were either a) 1 operating system or b) 1 installer / package mgmt route.
Right now, we have a mix of both and feel users will find it difficult to parse.
There was a problem hiding this comment.
ok yeah i just didn't want to repeat the curl instructions -- will update
There was a problem hiding this comment.
ok took a pass at reorganizing, let me know your thoughts c068e98
Test with:
curl -fsSL https://raw.githubusercontent.com/github/copilot-cli/refs/heads/devm33/install/install.sh | bash