Skip to content

fix: prevent double git init in quickstart command#91

Merged
Huerte merged 1 commit into
mainfrom
fix/new-double-init
Jun 12, 2026
Merged

fix: prevent double git init in quickstart command#91
Huerte merged 1 commit into
mainfrom
fix/new-double-init

Conversation

@Huerte

@Huerte Huerte commented Jun 12, 2026

Copy link
Copy Markdown
Owner

Type

  • Bug fix
  • New feature
  • Documentation
  • Refactor / internal

Overview

fixed a bug where gitgo new was quietly skipping the initial commit and push step. init_operation was calling git init, and then link_core was running it again. link_core saw it was already a repo and wrongly assumed it was just connecting an existing project to a remote, so it skipped pushing the newly scaffolded files.

Closes #

Changes

  • link.py: added an already_initialized parameter to link_core. when true, it skips calling git init and proceeds directly to creating the initial commit and pushing.
  • new.py: updated new_operation to pass already_initialized=True when it calls link_core.
  • test_new.py: updated the test assertions to expect the new parameter.
  • CHANGELOG.md: added an entry documenting the bug fix under the unreleased section.

How to Test

  1. pip install -e ".[dev]"
  2. gitgo new test-app python -p
  3. Expected result: it should now properly commit and push the scaffolded files to github instead of stopping at "Remote linked to existing repository".
  4. pytest tests/test_new.py -v
  5. Expected result: tests should pass successfully.

Checklist

  • I tested my changes locally and they work
  • I updated CHANGELOG.md under the [Unreleased] section
  • I updated README.md (if I added or changed a command)
  • I added or updated tests for my change (if applicable)
  • My change does not break any existing commands (if it does, describe the impact in the Overview)

@Huerte Huerte merged commit 3f5b453 into main Jun 12, 2026
1 check passed
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.

1 participant