Skip to content

Conversation

@OskarStark
Copy link
Contributor

No description provided.

Update Psalm from 5.18.0 to 6.13.1 for PHP 8.4 compatibility.
- Suppress MissingOverrideAttribute warnings
- Suppress UnusedClass warnings (for auto-wired classes like event subscribers)
- Clear unused baseline entries
- Update baseline version to 6.13.1
- Remove UnusedClass suppression from psalm.xml
- Generate baseline with all existing issues (UnusedClass, PossiblyUnusedMethod, UnusedProperty)
- This allows us to track new issues while ignoring existing ones
@OskarStark
Copy link
Contributor Author

OskarStark commented Nov 25, 2025

@stof are you fine by adding these things to the baseline or shall we suppress UnusedClass entirely?

- Use ensureOverrideAttribute="false" instead of suppressing MissingOverrideAttribute
- Use findUnusedCode="false" instead of baseline entries for unused code
- This is cleaner since we don't use @api annotations to define public APIs
- Clear baseline file as unused code checks are now disabled
@OskarStark OskarStark merged commit fb1c880 into symfony-tools:master Nov 26, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants