Skip to content

Fix stack 01: ignore secrets/reports, untrack committed run artifacts#25

Merged
ebarti merged 1 commit into
codex/adapter-hardeningfrom
review-fixes/01-hygiene
Jul 2, 2026
Merged

Fix stack 01: ignore secrets/reports, untrack committed run artifacts#25
ebarti merged 1 commit into
codex/adapter-hardeningfrom
review-fixes/01-hygiene

Conversation

@ebarti

@ebarti ebarti commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Stacked review-fix PR 1/N (base: `codex/adapter-hardening`).

From the deep review — repo hygiene, safety-first:

  • Add `.env` / `.env.*` to `.gitignore`. The working-tree `.env` holds a real `GH_TOKEN`; the `sdk_evolution_agent` example runs `git add`/`commit`/`push` autonomously, so an accidental commit was one stray `git add -A` away.
  • Untrack the previously committed `reports/sdk-evolution-autonomous/**` run artifacts (captured model I/O + local paths) and broaden the ignore to `reports/`.
  • Ignore local `REVIEW.md`.

No code changes. .env/REVIEW.md remain on disk, now ignored.

- Add .env, .env.* to .gitignore (the working tree .env holds a real
  GH_TOKEN and the sdk-evolution example runs git add/commit/push
  autonomously, so an accidental commit was one stray 'git add -A' away).
- Add REVIEW.md to local review-artifact ignores.
- Broaden the reports ignore to reports/ and untrack the previously
  committed reports/sdk-evolution-autonomous/** run artifacts (model I/O
  and local paths that should not live in version control).
@ebarti ebarti merged commit 6399de6 into codex/adapter-hardening Jul 2, 2026
8 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.

1 participant