Fix .gitattributes#7602
Open
Growl1234 wants to merge 2 commits into
Open
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adjusts repository export and normalization metadata by fixing the .gitattributes rule used to exclude the .github directory from git archive exports, and includes an incidental re-normalization of a Sphinx make.bat script due to attribute application.
Changes:
- Fix
.gitattributesexport-ignorepattern for the.githubdirectory. - Re-save
source/source_basis/module_ao/1_Documents/sphinx/make.batwith no functional content changes (diff appears attributable to attribute-driven normalization).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.gitattributes |
Updates export-ignore entry to correctly target .github for archive exports. |
source/source_basis/module_ao/1_Documents/sphinx/make.bat |
No functional changes; file content appears unchanged aside from normalization effects. |
ZhouXY-PKU
reviewed
Jul 7, 2026
Collaborator
Line ending of |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
.github/ export-ignoredoes not work withgit-archive-all, which checks parent directory paths without a trailing slash. Use.github export-ignoreinstead, which works for both Git andgit-archive-all.Note
make.batis changed due to an automatic application of.gitattributes; honestly I'm a bit confused because my change didn't affect any line break rules... And it now breaks my other branches, because this auto-change happens on every branch on my local machine but only commited in thisdevelopbranch. Might same as #7505?One additional concern is developers who have already run
git add --renormalizeon an older local branch. Even if they did not intentionally editmake.bat, that command may leave its normalized form staged in the index. A normalgit pull --rebasewill then refuse to start while such an uncommitted index change exists, so merging this PR alone will not unblock those branches: the local staged change remains until it is explicitly discarded, stashed, or committed. If it is committed merely to unblock the rebase, that commit may become redundant or empty once rebased onto a branch containing this normalization; it should only be skipped or dropped after confirming that it contains no real changes.This is therefore a reason to merge the normalization promptly, ideally as an isolated commit, to minimize the period during which active branches differ. It will not, however, automatically clean up normalization changes that developers have already created locally.
Details
Reminder
AGENTS.mdanddocs/developers_guide/agent_governance.md.source/changes.Linked Issue
None
Unit Tests and/or Case Tests for my changes
None
Exact Verification Performed
What's changed?
-See above
Governance Checklist
GlobalV,GlobalC, orPARAMcode references, or exception requested below with reason, scope, risk, and cleanup plan..hpppropagation, or rationale provided below..batand.cmduse CRLF.CMakeLists.txt, or rationale provided below.@coderabbitai review.INPUT Parameter Changes
docs/parameters.yamlupdated: nodocs/advanced/input_files/input-main.mdupdated: noCore Module Impact
Governance Exception