Skip to content

fix: prevent init from overwriting config#150

Merged
rokuosan merged 1 commit into
rokuosan:mainfrom
rokuosanai:fix/prevent-init-overwrite
Jul 24, 2026
Merged

fix: prevent init from overwriting config#150
rokuosan merged 1 commit into
rokuosan:mainfrom
rokuosanai:fix/prevent-init-overwrite

Conversation

@rokuosanai

Copy link
Copy Markdown
Contributor

Summary

  • make github-issue-cms init fail if gic.config.yaml already exists
  • add --force as the explicit opt-in to overwrite it
  • add regression coverage that verifies the existing file remains unchanged

Tests

  • TMPDIR="$PWD/.tmp/go-build" go test ./...
  • TMPDIR="$PWD/.tmp/go-build" go vet ./...

@rokuosan

Copy link
Copy Markdown
Owner

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 8d7315fdbe

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@rokuosan
rokuosan merged commit e3e1399 into rokuosan:main Jul 24, 2026
4 checks 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.

2 participants