Skip to content

Narrow rule complexity, add custom PHPStan to check whole rule complexity tree and spot architecture flaws#5025

Merged
TomasVotruba merged 4 commits intomainfrom
tv-file-providers
Sep 19, 2023
Merged

Narrow rule complexity, add custom PHPStan to check whole rule complexity tree and spot architecture flaws#5025
TomasVotruba merged 4 commits intomainfrom
tv-file-providers

Conversation

@TomasVotruba
Copy link
Copy Markdown
Member

@TomasVotruba TomasVotruba commented Sep 15, 2023

No description provided.

@TomasVotruba TomasVotruba marked this pull request as draft September 15, 2023 13:46
@staabm
Copy link
Copy Markdown
Contributor

staabm commented Sep 15, 2023

can't we already do the same thing with class-complexity rules?
https://github.com/TomasVotruba/cognitive-complexity

@TomasVotruba
Copy link
Copy Markdown
Member Author

We have those. This is slightly different and checks the peaks :)

@TomasVotruba TomasVotruba changed the title tv file providers Narrow rule complexity, add custom PHPStan to check whole rule complexity tree and spot architecture flaws Sep 19, 2023
@TomasVotruba TomasVotruba force-pushed the tv-file-providers branch 2 times, most recently from cfea725 to a3765f4 Compare September 19, 2023 12:20
@TomasVotruba TomasVotruba force-pushed the tv-file-providers branch 4 times, most recently from e1af418 to e0da377 Compare September 19, 2023 13:29
@TomasVotruba TomasVotruba merged commit 765e849 into main Sep 19, 2023
@TomasVotruba TomasVotruba deleted the tv-file-providers branch September 19, 2023 13:36
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.

2 participants