Commit graph

11279 commits

Author SHA1 Message Date
Babak K. Shandiz
3e946a356c
test(pr merge): always assert stderr
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-05 15:02:41 +01:00
Babak K. Shandiz
b1e1c8d500
test(pr merge): verify deleteRemoteBranch behaviour when API returns error
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-05 14:54:33 +01:00
Babak K. Shandiz
bafefb345e
fix(pr merge): ignore 404 as error when deleting remote branch
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-05 14:54:28 +01:00
William Martin
1cbfbf8eef
Merge pull request #11233 from cli/wm/bump-go-has-token
Inject token into bump-go workflow
2025-07-05 13:27:27 +02:00
William Martin
00cb1efe83 Ensure go mod tidy is run in bump-go
This is because go mod tidy seems to add minor version to the go mod
directive when it is missing.
2025-07-05 13:17:00 +02:00
William Martin
e29723d0f3 Inject token into bump-go workflow 2025-07-05 12:07:53 +02:00
Babak K. Shandiz
5ac184606f
Merge pull request #11172 from cli/babakks/use-job-run-log-api
Use Actions API to retrieve job run logs as a fallback mechanism
2025-07-04 16:34:47 +01:00
William Martin
7eef7af23d Reformat gh run view help 2025-07-04 17:25:59 +02:00
Babak K. Shandiz
0d8d330eb1
docs(run view): explain restrictions of fallback API calls
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-04 16:20:08 +01:00
Babak K. Shandiz
25ecbed620
test(run view): delete unused ZIP archive
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-04 14:13:08 +01:00
Babak K. Shandiz
53cf65e186
refactor(run view): remove Log field from DTO types
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-04 14:12:34 +01:00
Babak K. Shandiz
555b8f1bf9
test(run view): update tests
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-04 14:12:07 +01:00
Babak K. Shandiz
6d65904fee
refactor(run view): use API as fallback to fetch job logs
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-04 14:11:09 +01:00
Babak K. Shandiz
e0c7a328d2
test: add BinaryResponse helper function
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-04 14:09:09 +01:00
William Martin
0037f90f47
Merge pull request #11229 from cli/wm/fix-bump-go-git
Ensure bump go script has git user configured
2025-07-04 14:57:46 +02:00
William Martin
12b9865a90 Ensure bump go script has git user configured 2025-07-04 14:41:22 +02:00
William Martin
f5134c48c0
Merge pull request #11216 from tklauser/docs-install-linux-apt
docs: consistently use `apt` in installation instructions
2025-07-03 17:25:05 +02:00
William Martin
d64172256a
Merge pull request #11217 from cli/9808-gh-secret-set-using-selected-visibility-requires-a-list-of-repositories-its-optional-in-the-apibrowser
Support `--no-repos-selected` on `gh secret set`
2025-07-03 17:23:58 +02:00
William Martin
502b64582d Support --no-repos-selected on secret set 2025-07-03 17:02:28 +02:00
Tobias Klauser
fbb749613a docs: consistently use apt in installation instructions
Don't mix apt and apt-get, consistently use apt.
2025-07-03 15:16:07 +02:00
William Martin
5450b7f0ba
Merge pull request #11213 from cli/11207-consume-dependabot-minor-upgrades
Consume dependabot minor versions for go modules
2025-07-03 13:37:46 +02:00
William Martin
498ad84fcd Consume dependabot minor versions for go modules 2025-07-03 12:14:44 +02:00
William Martin
329dfa8ee7
Merge pull request #11205 from cli/wm/bump-dev-tunnels
Update microsoft dev-tunnels to v0.1.13
2025-07-03 12:08:40 +02:00
Babak K. Shandiz
3680a1fb22
Merge pull request #11192 from cli/babakks/decouple-arg-parsing-from-pr-finder
Decouple arg parsing from PR finder
2025-07-03 11:02:55 +01:00
Babak K. Shandiz
8ab5e84a12
test: add test for ParseURL
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-03 10:21:55 +01:00
Sam Coe
545b420ff8
Transform extension and filename qualifiers into path qualifier 2025-07-03 07:43:35 +02:00
Andy Feller
c69f828019 Initial assign Copilot during issue create changes
- updated command documentation for new `--assignee @copilot` syntax
- updated `gh issue create` logic with feature detection for actor availability
- fixed bug where `--assignee` would not select default assignees due to login vs display name mismatch
- updated survey prompt logic to avoid parsing prompt options in favor of correlating with assignee / actor source

One thing that is not included in these changes is incorporating the GraphQL mutation to replace assignees with actors, which I will continue iterating on.
2025-07-02 16:22:12 -04:00
William Martin
ad089d48d9 Update microsoft dev-tunnels to v0.1.13
https://github.com/microsoft/dev-tunnels/pull/322 introduced a few
breaking changes:
 * Port Tags were renamed to Labels
 * Client construction must now provide an API version (of which there is only one)
 * The /api/v1 prefix was dropped from request paths
 * TunnelPortListResponses may now be paginated (but we don't support that)
 * Requests to create a port with a changed protocol began erroring
2025-07-02 16:38:07 +02:00
William Martin
4c47bad604
Merge pull request #11203 from cli/wm/bump-deps
Bump all dependencies except dev-tunnels
2025-07-02 16:35:14 +02:00
William Martin
96956da8de Bump all dependencies except dev-tunnels 2025-07-02 14:56:58 +02:00
William Martin
2b84a3d698
Merge pull request #11194 from Stefan-Heimersheim/patch-1
Fix inconsistent use of tabs and spaces
2025-07-01 13:05:36 +02:00
Stefan Heimersheim
e062330163
Fix inconsistent use of tabs and spaces
The code below used spaces to indent two of the lines, and tabs for the other 6 lines. This makes indenting inconsistent in settings where tabs are not rendered as 8 spaces. I've replaced the spaces with tabs.

```
(type -p wget >/dev/null || (sudo apt update && sudo apt-get install wget -y)) \
	&& sudo mkdir -p -m 755 /etc/apt/keyrings \
        && out=$(mktemp) && wget -nv -O$out https://cli.github.com/packages/githubcli-archive-keyring.gpg \
        && cat $out | sudo tee /etc/apt/keyrings/githubcli-archive-keyring.gpg > /dev/null \
	&& sudo chmod go+r /etc/apt/keyrings/githubcli-archive-keyring.gpg \
	&& sudo mkdir -p -m 755 /etc/apt/sources.list.d \
	&& echo "deb [arch=$(dpkg --print-architecture) signed-by=/etc/apt/keyrings/githubcli-archive-keyring.gpg] https://cli.github.com/packages stable main" | sudo tee /etc/apt/sources.list.d/github-cli.list > /dev/null \
	&& sudo apt update \
	&& sudo apt install gh -y
```
2025-07-01 11:43:04 +01:00
Babak K. Shandiz
b7fa5ea7a9
docs: explain PR URL parsing reason
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-01 11:41:07 +01:00
Babak K. Shandiz
dc51226ce2
test: improve test case to highlight host name override
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-01 11:21:24 +01:00
William Martin
c58a88a217
Merge pull request #11189 from cli/wm/automate-go-bump
Add workflow to automate go version bumping
2025-07-01 12:21:01 +02:00
Babak K. Shandiz
58ed691a6c
test: remove references to AssignedActorsUsed field
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-01 11:19:13 +01:00
Babak K. Shandiz
2c86246a1a
test: verify providing a URL arg affects the base repo
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-01 11:18:38 +01:00
Babak K. Shandiz
b85dc94f84
refactor: select PR fields based on detected features
This change is almost a revert to what had already done in #10984.

Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-01 11:14:50 +01:00
Babak K. Shandiz
f65683784e
test: remove tests verifying assigne-related behaviour
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-01 11:12:58 +01:00
Babak K. Shandiz
12f4794151
fix: remove assignee-related intervention
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-01 11:12:16 +01:00
Babak K. Shandiz
00a86a105e
fix: remove AssignedActorsUsed field
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-01 11:11:28 +01:00
William Martin
12aeb1fed2 Add workflow to automate go version bumping 2025-07-01 12:10:22 +02:00
Babak K. Shandiz
23c9ff6d89
fix: expose ParseURL as a public func
Signed-off-by: Babak K. Shandiz <babakks@github.com>
2025-07-01 10:49:18 +01:00
Kynan Ware
729ff163b6
Merge pull request #11190 from cli/kw/followup-pr-11179
Remove unused `GH_TOKEN` env variable from workflow
2025-06-30 11:05:23 -06:00
Kynan Ware
8a5302ec6e Remove unused GH_TOKEN env variable from workflow
The GH_TOKEN environment variable was set but not used in the pr-vars-dispatch step. This commit removes it for clarity and to avoid confusion.
2025-06-30 10:55:10 -06:00
Kynan Ware
45c8c827c5
Add workflow_dispatch support to PR Help Wanted check (#11179)
* Add workflow_dispatch support to PR Help Wanted check

This update allows the PR Help Wanted workflow to be triggered manually via workflow_dispatch with a specified PR URL. It adds logic to fetch PR details using the GitHub CLI for manual runs and unifies variable handling for both event types.

* Update workflow to use PR number instead of URL

Changed the workflow_dispatch input from 'pr_url' to 'pr_number' and updated the script to construct the PR URL from the number.

* Move help-wanted check for draft PRs into script

* Don't prefix URL with `#`

* Invert draft checking logic

Inverting this logic because anything other than "false" means we should skip it.

* Move PR draft status check to shell script

The logic for checking if a pull request is a draft has been moved from the GitHub Actions workflow YAML to the check-help-wanted.sh script. This simplifies the workflow file and centralizes the draft status check within the script.
2025-06-30 12:51:54 -04:00
Andy Feller
e5913fe02f
Merge pull request #11038 from anuraaga/active-token-user
fix: get token for active user instead of blank if possible
2025-06-27 08:27:36 -04:00
Andy Feller
2fc1e548cc
Merge pull request #11048 from cli/babakks/embed-winres-at-release
Embed Windows resources (VERSIONINFO) during build
2025-06-24 17:29:18 -04:00
Babak K. Shandiz
9c54fb3e0d
chore: improve error message when versioninfo.json is not found
Co-authored-by: Andy Feller <andyfeller@github.com>
2025-06-24 22:19:37 +01:00
Daniel Krzeminski
b31f38c946
Fix: gh pr create prioritize --title and --body over --fill when --web is present (#10547)
* fix: prioritize title and body over autofill for prs

* fix: prioritize title and body over autofill tests

* refactor: collapse state conditions

---------

Co-authored-by: Babak K. Shandiz <babakks@github.com>
2025-06-24 12:54:43 +00:00