fix: add CLAUDE.md to vendor excludes

[williamdes]

No description provided.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.72%. Comparing base (11485d8) to head (356b3b9).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##               main     #236   +/-   ##
=========================================
  Coverage     71.72%   71.72%           
  Complexity      999      999           
=========================================
  Files            49       49           
  Lines          3158     3158           
=========================================
  Hits           2265     2265           
  Misses          893      893           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[DASPRiD]

Thanks! I really wish we'd have a whitelist instead of a blacklist way, just like package.json has… :)

[williamdes]

Thanks! I really wish we'd have a whitelist instead of a blacklist way, just like package.json has… :)

.gitignore has this by using

/
!/keepme

maybe gitattributes has it too
But you would have to do a self-test to ensure it is up to date.
ask for a git archive, keep tests from the checkout
manage to build an autoloader for both
and run tests

Welcome to what we have to do in Debian:

• https://salsa.debian.org/php-team/pear/php-bacon-baconqrcode/-/blob/debian/3.0.4-1/debian/rules?ref_type=tags
• https://salsa.debian.org/php-team/pear/php-bacon-baconqrcode/-/blob/debian/3.0.4-1/debian/tests/control?ref_type=tags (test the installed package)

[williamdes]

By the way at phpMyAdmin I built a script to find such dev files and add them to an exclude list: https://github.com/phpmyadmin/phpmyadmin/blob/QA_5_2/scripts/check-release-excludes.sh

[DASPRiD]

@williamdes Hm, I suppose I could introduce a simple github workflow step which validates that a git archvie tar only includes composer.json, LICENSE, README.md and the src folder.

I suppose that should be safe?

[williamdes]

@williamdes Hm, I suppose I could introduce a simple github workflow step which validates that a git archvie tar only includes composer.json, LICENSE, README.md and the src folder.

I suppose that should be safe?

Absolutely
My script allows to do that in some ways
But see how I auto test Doctum: https://github.com/code-lts/doctum/blob/main/tests/composer-package-auto-test.sh

It is really important to run tests against the archive