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

Add --strict-psr flag to dump-autoload to fail the process if psr violations were detected #10886

Merged
merged 1 commit into from Jun 20, 2022

Conversation

Seldaek
Copy link
Member

@Seldaek Seldaek commented Jun 19, 2022

fixes #10241

@Seldaek Seldaek merged commit e853069 into composer:main Jun 20, 2022
@Seldaek Seldaek deleted the validate_psr branch June 20, 2022 11:57
emahorvat52 pushed a commit to emahorvat52/composer that referenced this pull request Feb 3, 2023
@mvorisek
Copy link
Contributor

mvorisek commented Jul 9, 2023

@Seldaek In our CI we use composer validate --strict to validate composer file, what about supporting --strict-psr flag in composer validate command as well?

@Seldaek
Copy link
Member Author

Seldaek commented Jul 18, 2023

IMO you rather do dump with --strict-psr on top of it if you want that, because it requires an optimized dump and right now validate doesn't really do any dumping at all, so not sure it'd fit in there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Introduce an option of dump-autoload to fail on PSR uncompliant classes
2 participants