Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix initial commit on fresh install #544

Merged
merged 1 commit into from
Oct 7, 2022

Conversation

sbkok
Copy link
Collaborator

@sbkok sbkok commented Oct 7, 2022

Why?

When the repository is created, there are no commits in the repository yet. The process tried to create a branch where the parent commit id was empty. This is not allowed.

What?

Instead, it should only create the branch when there is a commit to branch off from. Otherwise it should only create the new commits on the main branch directly.


By submitting this pull request, I confirm that you can use, modify, copy, and
redistribute this contribution, under the terms of your choice.

**Why?**

When the repository is created, there are no commits in the repo yet.
The process tried to create a branch where the parent commit id was empty.
This is not allowed.

**What?**

Instead, it should only create the branch when there is a commit to branch off
from. Otherwise it should only create the new commits on the main branch
directly.
@sbkok sbkok added the bug Something isn't working label Oct 7, 2022
@sbkok sbkok added this to the v3.2.0 milestone Oct 7, 2022
@sbkok sbkok requested review from javydekoning and StewartW October 7, 2022 07:36
@sbkok sbkok merged commit 11317d4 into awslabs:master Oct 7, 2022
@sbkok sbkok deleted the fix/initial-commit-fresh-install branch October 7, 2022 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants