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 ClassMap@getRawPsrViolations() #16

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

cosmastech
Copy link
Contributor

Motivation

I'd like to build a step in our gitlab pipeline that confirms that all files meet the PSR-4 namespace standard.

How it can be solved now

  • Use reflection to extract the $psrViolations from ClassMap, or
  • Attempt to parse the strings from the warnings returned from ClassMap@getPsrViolations()

Changes

Adds a method to get the raw violations (ClassMap@getRawPsrViolations()) and add tests.

Unverified

This user has not yet uploaded their public signing key.
@Seldaek
Copy link
Member

Seldaek commented Nov 25, 2024

You should be able to use composer dump-autoload --strict-psr --optimize to achieve this - instead of having to script some custom code using this lib.

But that said, I don't see why it shouldn't be possible, so I guess I'm fine merging this. Thanks

@Seldaek Seldaek merged commit 9ab27fb into composer:main Nov 25, 2024
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.

None yet

2 participants