Repository navigation
fix: preserve consumer Git rule precedence - #392
Conversation
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 25 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Changelog✅ Contribution check passed. Head: Base: Contribution: 1 new valid fragment(s). Policy: ordinary.
|
|
🚀 Preview is available for this pull request. |
Related Issues
Closes #378
Closes #385
Changes
.gitignorepattern order, repeated patterns and significant whitespace. Prepend canonical defaults once instead of sorting rules and undoing negations..gitattributesrule order, repetitions, quoted paths and unset/unspecified overrides. Prepend only missing generated defaults; preserve explicit keep-in-export filtering.0.Verification
important.logremains not ignored andkeep.txtretains export-ignore=unset after synchronization.git diff --checkPASS. Independent test-guardian review found/fixed literal0and glob anchoring regressions.composer dev-toolsunavailable locally due missing dependencies; prior dependency download denial respected. Hosted CI must validate full suite.Documentation / Generated Output
Updated GitIgnore/GitAttributes command docs and merger contracts. No consumer files synchronized.
Changelog
Added
.changelog/git-rule-precedence.md(fixed/patch).Reviewer Notes
Focused correctness fixes, not extraction. Existing history and manual rules retained; no merge, release, or PR #376 changes.