Skip to content

fix(.gitignore): align logging managed block with setup.conf (#115)#116

Merged
ycpss91255 merged 1 commit into
mainfrom
fix/115-gitignore-stale-logs
May 26, 2026
Merged

fix(.gitignore): align logging managed block with setup.conf (#115)#116
ycpss91255 merged 1 commit into
mainfrom
fix/115-gitignore-stale-logs

Conversation

@ycpss91255
Copy link
Copy Markdown
Contributor

Summary

Closes #115. Fixes the stale /logs/ entry in the [logging] local_path managed marker block of .gitignore.

PR #114 upgraded .base to v0.34.0 and flipped [logging] local_path
from ./logs/ to ./log/ in config/docker/setup.conf, but the
managed block in .gitignore was not updated because setup.sh apply
did not run after the config change. The v0.34.0 prune mechanism
(base#390) would have fixed this on the next wrapper invocation, but
the stale entry persisted in the meantime.

Audit findings (issue #115)

Full audit of .gitignore against base's canonical output:

  • All 8 canonical entries present and correct.
  • No tracked files that should be untracked.
  • Repo-specific entries (bridge.yaml, .claude, CLAUDE.md) valid.
  • Only finding: /logs/ stale in managed block (fixed by this PR).

Test plan

  • Single-line .gitignore change: /logs/ -> /log/.
  • CI green (ci-summary aggregator).

Context

The root cause (two-source gitignore lifecycle: init/upgrade vs
runtime) is tracked in ycpss91255-docker/base#402.

…th (#115)

PR #114 flipped [logging] local_path from ./logs/ to ./log/ in
setup.conf but the managed marker block in .gitignore was not
updated because setup.sh apply did not run after the config change.

Closes #115
@ycpss91255 ycpss91255 enabled auto-merge (squash) May 26, 2026 00:35
@ycpss91255 ycpss91255 merged commit 0133272 into main May 26, 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.

audit .gitignore + generated artifacts against base canonical output

1 participant