Skip to content

Issue 4574: Allow pull on empty repo#21528

Merged
niik merged 3 commits intodesktop:developmentfrom
jackfreem:jf/Issue4574
Mar 5, 2026
Merged

Issue 4574: Allow pull on empty repo#21528
niik merged 3 commits intodesktop:developmentfrom
jackfreem:jf/Issue4574

Conversation

@jackfreem
Copy link
Contributor

@jackfreem jackfreem commented Jan 28, 2026

Closes #4574
Closes #4575

Description

  • Allows origin fetch for fully empty repo

Screenshots

Before:
Screenshot 2026-01-27 at 7 47 27 PM

After:
Screenshot 2026-01-27 at 7 47 35 PM

Release notes

Notes: Bug fix: Allow origin fetch on empty repo

@github-actions github-actions bot added external Pull request originating outside of the Desktop core team triage For incoming issues labels Jan 28, 2026
@jackfreem jackfreem marked this pull request as ready for review January 28, 2026 00:51
Copy link
Member

@niik niik left a comment

Choose a reason for hiding this comment

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

Hey @jackfreem, thanks for wanting to contribute. I think these changes are sound but there's one interesting logical consequence of your changes that I'd like you to take a look at

@niik niik self-assigned this Feb 19, 2026
@niik niik removed the triage For incoming issues label Feb 19, 2026
Since remoteName === null already returns early, the null check inside
the TipState.Unborn block was always true. Simplify to call fetchButton
directly and remove the now-unreachable unbornRepositoryButton method.
@jackfreem jackfreem requested a review from niik February 23, 2026 00:31
Copy link
Member

@niik niik left a comment

Choose a reason for hiding this comment

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

Thanks @jackfreem!

@niik niik merged commit c523f08 into desktop:development Mar 5, 2026
7 checks passed
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 Desktop core team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot pull after cloning a truly empty repository

2 participants