Skip to content

Update .gitattributes to exclude dev files from composer dist#39

Open
williamdes wants to merge 1 commit into
phar-io:masterfrom
williamdes:williamdes/gitattributes-export-ignore
Open

Update .gitattributes to exclude dev files from composer dist#39
williamdes wants to merge 1 commit into
phar-io:masterfrom
williamdes:williamdes/gitattributes-export-ignore

Conversation

@williamdes

@williamdes williamdes commented May 20, 2026

Copy link
Copy Markdown

This avoids shipping dev/tooling files in the composer dist that aren't needed at runtime.

Last .gitattributes update was 3d6d988 (2024-03-12, #36).

Stale entry to remove

  • /.php_cs.dist.php export-ignore - the old PHP CS Fixer 2.x config filename. The repo renamed it to .php-cs-fixer.dist.php in a9fa919 (2022-02-17), and the new name is added below.

Entry to add

  • /.php-cs-fixer.dist.php export-ignore - last touched in a9fa919 (2022-02-17). Still shipped in the composer dist.

Also realigned the column width to fit /.php-cs-fixer.dist.php (the longest path now).

Background reading: https://blog.madewithlove.be/post/gitattributes/

This avoids shipping dev/tooling files in the composer dist that aren't needed at runtime.

Last `.gitattributes` update was 3d6d988 (2024-03-12, phar-io#36).

### Stale entry to remove
- `/.php_cs.dist.php export-ignore` - the old PHP CS Fixer 2.x config filename. Renamed to `.php-cs-fixer.dist.php` since v3, and the new name is also being added in this PR.

### Entry to add
- `/.php-cs-fixer.dist.php export-ignore` - last touched in a9fa919 (2022-02-17). Still shipped in the composer dist.

Also realigned the column width on every line for the longest path (`/.php-cs-fixer.dist.php`).

Background reading: https://blog.madewithlove.be/post/gitattributes/

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@williamdes
williamdes force-pushed the williamdes/gitattributes-export-ignore branch from 69da504 to 2df92eb Compare May 22, 2026 15:53
@williamdes

Copy link
Copy Markdown
Author

Do you want to exclude the changelog file too ?

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