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

Bugfix: Ignore the 'default_branch' option #52

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

scoopex
Copy link
Contributor

@scoopex scoopex commented Nov 20, 2024

When specifying a default branch on a repository which is still empty, the execution fails with the following error message:
'Cannot update default branch for an empty repository. Please init the repository and push first'

This change avoids that users have to debug this problem and/or create new repositories in three steps (1. ‘create repo PR’, 2. add branch , 3. ‘set default branch’ PR).

Secondly this change adds a bit more detailed error-debug-output to simplify
debugging situations.

When specifying a default branch on a repository which is still empty, the execution
fails with the following error message:
'Cannot update default branch for an empty repository. Please init the repository and push first'

This change avoids that users have to debug this problem and/or create new repositories in
three steps (1. ‘create repo PR’, 2. add branch , 3. ‘set default branch’ PR).

Secondly this change adds a bit more detailed error-debug-output to simplify
debugging situations.

Signed-off-by: Marc Schöchlin <[email protected]>
Signed-off-by: Marc Schöchlin <[email protected]>
Signed-off-by: Marc Schöchlin <[email protected]>
@scoopex
Copy link
Contributor Author

scoopex commented Jan 4, 2025

@gtema The pull request has been open for a while. I know that you are very busy. However, I don't want to pay endless attention to this PR and will close/remove it if I don't receive any feedback by 2025-01-31.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants