Skip to content

fix: correctly set ErrNotOnAnyBranch error in git client#7082

Merged
samcoe merged 1 commit intocli:trunkfrom
alex-petrov-vt:iss-7051
Mar 5, 2023
Merged

fix: correctly set ErrNotOnAnyBranch error in git client#7082
samcoe merged 1 commit intocli:trunkfrom
alex-petrov-vt:iss-7051

Conversation

@alex-petrov-vt
Copy link
Contributor

@alex-petrov-vt alex-petrov-vt commented Mar 3, 2023

Before, git client was not correctly reporting ErrNotOnAnyBranch error. Because of that, gh pr status was not functioning correctly when run from the detached HEAD. This PR fixes that.

Fixes #7051

Before, git client was not correctly setting `ErrNotOnAnyBranch` error.
Because of that, `gh pr status` was not functioning correctly when run
from the detached HEAD. This commit fixes that.

Fixes cli#7051
@alex-petrov-vt alex-petrov-vt requested a review from a team as a code owner March 3, 2023 20:57
@alex-petrov-vt alex-petrov-vt requested review from samcoe and removed request for a team March 3, 2023 20:57
@cliAutomation cliAutomation added the external pull request originating outside of the CLI core team label Mar 3, 2023
@samcoe samcoe self-assigned this Mar 5, 2023
Copy link
Contributor

@samcoe samcoe left a comment

Choose a reason for hiding this comment

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

@alex-petrov-vt Thanks for the fix!

@samcoe samcoe merged commit 7d4ab65 into cli:trunk Mar 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external pull request originating outside of the CLI core team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

gh pr status does not show PRs summary after upgrade when on detached HEAD

3 participants