-
Notifications
You must be signed in to change notification settings - Fork 138
Add PHPStan strict rules (except for empty.notAllowed) #1241
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
94f4f22
Add phpstan-strict-rules
westonruter 211d192
Ignore staticMethod.dynamicCall, booleanNot.exprNotBoolean, and if.co…
westonruter f2c5120
Fix cast.useless
westonruter 0c5c42b
Fix method.childReturnType
westonruter 66dd4d6
Fix plus.leftNonNumeric
westonruter db11733
Disable strictArrayFilter rule
westonruter 165b15b
Fix variable.implicitArray
westonruter d3e0f22
Fix instance of booleanAnd.leftNotBoolean
westonruter db82f1e
Update phpstan to 1.11.4
westonruter 7de2d96
Fix interface.nameCase
westonruter 87ecaad
Fix booleanNot.exprNotBoolean
westonruter 44d8375
Fix ternary.condNotBoolean
westonruter 94884c5
Fix booleanOr.rightNotBoolean
westonruter 5b5cd68
Fix booleanAnd.leftNotBoolean
westonruter dad5312
Fix if.condNotBoolean
westonruter 228035f
Ignore empty.notAllowed temporarily
westonruter 831105c
Treat a boolean src attribute as if it is an inline script
westonruter b99c7c0
Merge branch 'trunk' of https://github.com/WordPress/performance into…
westonruter 0413c60
Fix remaining strict rule violations
westonruter 350ddbf
Merge branch 'trunk' of https://github.com/WordPress/performance into…
westonruter d4af3a6
Fix and improve conditions in auto_sizes_filter_image_tag and add typing
westonruter 8e90f62
Remove unnecessary casting to int
westonruter 4f4f6bd
Remove unnecessary casting
westonruter eaf1f65
Fix logic inversion
westonruter File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Ignore empty.notAllowed temporarily
- Loading branch information
commit 228035ffa4805976d573d2af8dbdf1f2f4f6de30
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure why this issue is being reported for assertion method calls in tests.