Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix autoload configs to avoid warnings when building optimized autoloaders #57275

Merged
merged 1 commit into from
Jun 1, 2024

Conversation

Seldaek
Copy link
Member

@Seldaek Seldaek commented May 31, 2024

Q A
Branch? 5.4
Bug fix? yes
New feature? no
Deprecations? no
Issues Fix #...
License MIT

The next composer release next week (you can try composer dump -o with snapshots right now) warns a bit more about classes which do not match the psr-4 rules they are found in.. Symfony has quite a few false positives there because the rules are using "" as path for all individual components so all files of the component are scanned by default. I already made sure Composer excludes the vendor dir by default now, and this fixes the rest of the errors I noticed.

@fabpot
Copy link
Member

fabpot commented Jun 1, 2024

Thank you @Seldaek.

@fabpot fabpot merged commit 75ddcbb into symfony:5.4 Jun 1, 2024
12 of 13 checks passed
fabpot added a commit that referenced this pull request Jun 1, 2024
…ized autoloaders (Seldaek)

This PR was merged into the 7.1 branch.

Discussion
----------

Fix autoload configs to avoid warnings when building optimized autoloaders

…aders

| Q             | A
| ------------- | ---
| Branch?       | 7.1
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->
| License       | MIT

Same as #57275 but for 7.1 as the new Emoji component also needed a fix.

Commits
-------

68f6426 Fix autoload configs to avoid warnings when building optimized autoloaders
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants