Skip to content

[pull] main from github:main#37

Open
pull[bot] wants to merge 53 commits intoRodnyPHP:mainfrom
github:main
Open

[pull] main from github:main#37
pull[bot] wants to merge 53 commits intoRodnyPHP:mainfrom
github:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Feb 20, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

SteveSandersonMS and others added 3 commits February 20, 2026 09:49
Re-applies the change from PR #1497 which was reverted in PR #1548.

The original broke the workflow YAML because the PowerShell here-string
content and closing tag had zero indentation, which terminated the YAML
literal block scalar (run: |). Fixed by indenting the here-string body
to match the block indentation level; YAML strips the leading spaces,
leaving valid PowerShell with the closing tag at column 0.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Include PowerShell dependency in winget manifest (v3)
@pull pull bot locked and limited conversation to collaborators Feb 20, 2026
@pull pull bot added the ⤵️ pull label Feb 20, 2026
github-actions bot and others added 25 commits February 20, 2026 23:32
Updated installation instructions and removed npm references.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
When copilot is not on PATH after install, the script now:
- Tells users to restart their shell or source their RC file after
  adding the PATH export
- Provides a copy-pasteable one-liner (export PATH + copilot help)
  that fixes PATH for the current session and runs the getting started
  command
- Only shows the simple 'copilot help' message when it will actually
  work

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
When the GITHUB_TOKEN environment variable is set, pass it as an
Authorization header on curl/wget downloads and embed it in the git
remote URL for ls-remote. This helps avoid rate limits and enables
installation from private repositories.

When GITHUB_TOKEN is unset, behavior is unchanged — the auth arrays
expand to nothing and the git remote URL stays unauthenticated.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
install: use GITHUB_TOKEN for authenticated GitHub requests
The installer was adding PATH exports to .bashrc/.zshrc, which are
rc files for interactive shells.  PATH belongs in login-shell startup
files (~/.profile, ~/.bash_profile, ~/.zprofile) so it is set once
per session and inherited by all child processes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Alter PATH for login shells, not interactive shells
github-actions bot and others added 25 commits March 16, 2026 22:44
Tags are sorted lexicographically by default, not by version,
so for example right now `git ls-remote` for this repo shows:

```
$ git ls-remote --tags \
  https://github.com/github/copilot-cli \
  | tail -3 | awk -F/ '{print$NF}'
v1.0.8
v1.0.8-0
v1.0.9
```

With the sort option, it shows the correct latest prerelease tags:

```
$ git ls-remote --tags --sort "version:refname" \
  https://github.com/github/copilot-cli \
  | tail -3 | awk -F/ '{print$NF}'
v1.0.12-0
v1.0.12-1
v1.0.12-2
```

This option for git ls-remote was added in git version 2.18.0 in June 2018:
https://github.com/git/git/blob/v2.18.0/Documentation/RelNotes/2.18.0.txt#L69-L70
Sort git tags when determining prerelease version
Fish shell users currently fall into the catch-all case, which writes
POSIX export syntax to ~/.profile. Fish does not source ~/.profile and
does not use export PATH="...:$PATH" syntax, so the PATH addition
silently does nothing.

Add a fish case that targets the idiomatic conf.d directory and uses
fish_add_path, matching how the script already handles zsh and bash
with their respective profile files and syntax. Extract the PATH
command into a variable to avoid duplicating the shell-specific logic
across the interactive prompt, non-interactive hint, and get-started
instructions.
Replace scattered rm -rf calls with a single EXIT trap to ensure the
temp directory is always cleaned up on exit. Previously, failures in
curl/wget download, tar extraction, chmod, or mkdir left temp files
behind because set -e would exit before reaching manual cleanup calls.

The trap fires on any exit (success or failure), so it covers all
paths with one line while removing five redundant cleanup calls.
install: use EXIT trap for temp directory cleanup
install: add fish shell support for PATH configuration
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants