Skip to content

fix: exclude dev files from Composer distribution archive#53

Merged
voku merged 1 commit intomasterfrom
copilot/fix-installing-voku-stringy
Apr 24, 2026
Merged

fix: exclude dev files from Composer distribution archive#53
voku merged 1 commit intomasterfrom
copilot/fix-installing-voku-stringy

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 24, 2026

Installing voku/stringy:7.0.0 failed with a corrupt/unextractable zip because dev-only files introduced in this release cycle were missing from .gitattributes export-ignore, while phpunit.xml.dist was already excluded — an asymmetry that can confuse extraction on case-insensitive filesystems.

Changes

  • .gitattributes — add missing export-ignore entries:

    • /phpunit.legacy.xml.dist (added alongside phpunit.xml.dist but never excluded)
    • /.deepsource.toml
    • /phpcs.php_cs
  • CHANGELOG.md — remove incorrect "breaking-change: drop support for PHP 7.1" (the release actually re-adds PHP 7.1+ support per composer.json: "php": ">=7.1.0"); add entry noting the distribution fix and PHP 8.1 JsonSerializable shim.

Distribution archive shrinks from 14 → 11 files, containing only src/, composer.json, README.md, LICENSE.txt, and CHANGELOG.md.


This change is Reviewable

@voku voku marked this pull request as ready for review April 24, 2026 02:18
@voku voku merged commit cb83848 into master Apr 24, 2026
11 of 15 checks passed
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 97.891%. remained the same — copilot/fix-installing-voku-stringy into master

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.

3 participants