cli: Feedback issue for bugs & inconsistencies with GitHub CLI

This is an issue to collect feedback about things that may not be working correctly or as expected with GitHub CLI. It’s not a place for feature requests, as we’d like our primary product feedback to be from users outside of GitHub. Additionally, this repository will likely become public so please don’t share any internally confidential information about upcoming GitHub things in here. Thanks!

To download GitHub CLI for macOS, run brew install github/gh/gh in your Terminal.

About this issue

  • Original URL
  • State: closed
  • Created 5 years ago
  • Comments: 16 (12 by maintainers)

Commits related to this issue

Most upvoted comments

Hi friends 👋 I got a similar panic to @shawnbot running gh pr status on http://github.com/github/ce-stardust:

Current branch
  There is no pull request associated with [master]

Created by you
  You have no open pull requests

Requesting a code review from you
  #934  [Security] Bump https-proxy-agent from 2.2.2 to... [dependabot/npm_and_yarn/frontend/https-proxy-agent-2.2.4]
   - checks: 11
  #927  Bump @craco/craco from 5.6.1 to 5.6.2 in /frontend [dependabot/npm_and_yarn/frontend/craco/craco-5.6.2]
   - checks: 9/10
  #926  Bump graphql-markdown from 5.1.0 to 5.2.0 in /f... [dependabot/npm_and_yarn/frontend/graphql-markdown-5.2.0]
   - checks: 9/10
  #925  Bump eslint-config-prettier from 6.5.0 to 6.6.0... [dependabot/npm_and_yarn/frontend/eslint-config-prettier-6.6.0]
   - checks: 10
  #924  Bump size-limit from 2.2.0 to 2.2.1 in /frontend [dependabot/npm_and_yarn/frontend/size-limit-2.2.1]
   - checks: 9/10
  #923  Bump @types/node from 12.12.7 to 12.12.8 in /fr... [dependabot/npm_and_yarn/frontend/types/node-12.12.8]panic: unsupported status: ""

goroutine 1 [running]:
github.com/github/gh-cli/api.(*PullRequest).ChecksStatus(0xc0005159c0, 0xc000096008, 0x14dbde2, 0xb, 0xc000515990)
	/home/runner/work/gh-cli/gh-cli/api/queries.go:101 +0x594
github.com/github/gh-cli/command.printPrs(0xc0000d0000, 0xa, 0x10)
	/home/runner/work/gh-cli/gh-cli/command/pr.go:358 +0x3a9
github.com/github/gh-cli/command.prStatus(0x1866060, 0x1888568, 0x0, 0x0, 0x0, 0x0)
	/home/runner/work/gh-cli/gh-cli/command/pr.go:101 +0x36e
github.com/spf13/cobra.(*Command).execute(0x1866060, 0x1888568, 0x0, 0x0, 0x1866060, 0x1888568)
	/home/runner/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:826 +0x460
github.com/spf13/cobra.(*Command).ExecuteC(0x18653e0, 0xc0000fdf20, 0x103aeda, 0x1827300)
	/home/runner/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:914 +0x2fb
main.main()
	/home/runner/work/gh-cli/gh-cli/main.go:12 +0x31

panic: unsupported status: "" seems like the bad part here 😅

@shawnbot @joshaber: Sorry about the crash! It’s since been fixed: brew upgrade gh

@leereilly @tierninho: Much appreciated 🙇

This is great!! I ran into one problem listing statuses for https://github.com/github/github/pull/127437:

$ git symbolic-ref
refs/heads/proper-tag-nesting

$ gh pr status
Current branch
  #129031  Enforce proper HTML tag nesting via erblint [proper-tag-nesting]
panic: unsupported check conclusion: ""

goroutine 1 [running]:
github.com/github/gh-cli/api.(*PullRequest).ChecksStatus(0xc0004199a0, 0xc00000e018, 0x14db07b, 0xb, 0xc000419970)
        /home/runner/work/gh-cli/gh-cli/api/queries.go:128 +0x695
github.com/github/gh-cli/command.printPrs(0xc000419c78, 0x1, 0x1)
        /home/runner/work/gh-cli/gh-cli/command/pr.go:358 +0x37d
github.com/github/gh-cli/command.prStatus(0x1865060, 0x1887568, 0x0, 0x0, 0x0, 0x0)
        /home/runner/work/gh-cli/gh-cli/command/pr.go:84 +0x270
github.com/spf13/cobra.(*Command).execute(0x1865060, 0x1887568, 0x0, 0x0, 0x1865060, 0x1887568)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:826 +0x460
github.com/spf13/cobra.(*Command).ExecuteC(0x18643e0, 0xc0000f7f20, 0x103aeda, 0x1826300)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:914 +0x2fb
main.main()
        /home/runner/work/gh-cli/gh-cli/main.go:12 +0x31

I’m going to close this now since we’re using individual issues to catalog things that come up now.

@seejohnrun thanks for trying it out! gh pr list will show you all open PRs in a repo (up to 30). You should try gh pr status it might be closer to what you’re looking for 😄. If there’s language you think would clarify all this would love to hear suggestions!

I’d also love a way to get a list similar to /pulls or just my created or assigned PRs ❤️

👋 Just trying this out - I was a bit confused by gh pr list which seems to contain a bunch of PRs I’ve never seen before. Maybe it’s worth adding a note about what this list contains in the output?

See #78 is missing…

@tierninho The “recent” section of issue status output right now only displays issues that were created or updated within last 24h. I think that at the time that you were looking at issue status, #78 was already more than a day old. Today it received new labels so it got bumped up to make the list again. We can have a discussion about whether that is correct behavior, but based on current functionality, I don’t think what you’ve seen was a bug.