Skip to content

Fix Windows asset URL in copilot command tests#12500

Merged
BagToad merged 1 commit intotrunkfrom
babakks/polish-copilot-cmd-tests
Jan 20, 2026
Merged

Fix Windows asset URL in copilot command tests#12500
BagToad merged 1 commit intotrunkfrom
babakks/polish-copilot-cmd-tests

Conversation

@babakks
Copy link
Member

@babakks babakks commented Jan 20, 2026

This is a quick one to fix the Windows asset URL/name in copilot command tests.

Signed-off-by: Babak K. Shandiz <babakks@github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes an incorrect Windows asset filename in the copilot command tests. The test was using "copilot-windows-x64.zip" but the actual implementation uses "copilot-win32-x64.zip" (as Windows is mapped to "win32" for asset naming).

Changes:

  • Updated test asset name from "copilot-windows-x64.zip" to "copilot-win32-x64.zip" in the fetchExpectedChecksum test

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@BagToad BagToad merged commit 3d89222 into trunk Jan 20, 2026
19 checks passed
@BagToad BagToad deleted the babakks/polish-copilot-cmd-tests branch January 20, 2026 16:39
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Jan 22, 2026
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [cli/cli](https://github.com/cli/cli) | minor | `v2.85.0` → `v2.86.0` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>cli/cli (cli/cli)</summary>

### [`v2.86.0`](https://github.com/cli/cli/releases/tag/v2.86.0): GitHub CLI 2.86.0

[Compare Source](cli/cli@v2.85.0...v2.86.0)

#### :copilot:  Install and run GitHub Copilot CLI directly from `gh`

Since we deprecated the [GitHub Copilot in the CLI](https://github.blog/changelog/2025-09-25-upcoming-deprecation-of-gh-copilot-cli-extension/) extension in favor of the new agentic [GitHub Copilot CLI](https://github.com/github/copilot-cli), we want to give developers using `gh` a simple way to get started using our most powerful terminal assistant.

- `gh copilot` will prompt to install, then run Copilot CLI
- `gh copilot <args>` will execute the Copilot CLI, forwarding any arguments and flags

For more information and usage options, run `gh copilot --help`.

#### What's Changed

##### ✨ Features

- `gh copilot`: add native `copilot` command to execute/install copilot cli by [@&#8203;devm33](https://github.com/devm33) in [#&#8203;12444](cli/cli#12444)
- `gh cache delete`: allow for delete all caches for a ref by [@&#8203;davidspek](https://github.com/davidspek) in [#&#8203;12101](cli/cli#12101)
- `gh pr create`: error when head and base refs are identical in pr create by [@&#8203;majiayu000](https://github.com/majiayu000) in [#&#8203;12376](cli/cli#12376)

##### 📚 Docs & Chores

- Fix Windows asset URL in `copilot` command tests by [@&#8203;babakks](https://github.com/babakks) in [#&#8203;12500](cli/cli#12500)
- Update contributing guidelines for clarity by [@&#8203;BagToad](https://github.com/BagToad) in [#&#8203;12505](cli/cli#12505)

#### New Contributors

- [@&#8203;devm33](https://github.com/devm33) made their first contribution in [#&#8203;12444](cli/cli#12444)
- [@&#8203;davidspek](https://github.com/davidspek) made their first contribution in [#&#8203;12101](cli/cli#12101)

**Full Changelog**: <cli/cli@v2.85.0...v2.86.0>

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0Mi44Ni4xIiwidXBkYXRlZEluVmVyIjoiNDIuODYuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiUmVub3ZhdGUgQm90IiwiYXV0b21hdGlvbjpib3QtYXV0aG9yZWQiLCJkZXBlbmRlbmN5LXR5cGU6Om1pbm9yIl19-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants