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

Improve PHPStan config #563

Merged
merged 5 commits into from
Jul 14, 2021
Merged

Improve PHPStan config #563

merged 5 commits into from
Jul 14, 2021

Conversation

westonruter
Copy link
Collaborator

@westonruter westonruter commented Jul 13, 2021

Improving config based on #562 props @szepeviktor.

Amends #560 for #243

@westonruter westonruter added this to the 0.7 milestone Jul 13, 2021
@google-cla google-cla bot added the cla: yes label Jul 13, 2021
@codecov-commenter
Copy link

codecov-commenter commented Jul 13, 2021

Codecov Report

Merging #563 (9d10a31) into develop (a1f9125) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop     #563   +/-   ##
==========================================
  Coverage      17.28%   17.28%           
  Complexity       296      296           
==========================================
  Files             55       55           
  Lines           1770     1770           
==========================================
  Hits             306      306           
  Misses          1464     1464           
Flag Coverage Δ
php 17.28% <ø> (ø)
unit 17.28% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a1f9125...9d10a31. Read the comment docs.

@szepeviktor
Copy link
Contributor

szepeviktor commented Jul 13, 2021

@westonruter CI is not configured to trigger static analysis on PHPStan configuration file change!

@@ -5,28 +5,17 @@ parameters:
level: 5
inferPrivatePropertyTypeFromConstructor: true
paths:
- %currentWorkingDirectory%/pwa.php
- %currentWorkingDirectory%/wp-admin
- %currentWorkingDirectory%/wp-includes
- %currentWorkingDirectory%/bundled-theme-support
- %currentWorkingDirectory%/integrations
bootstrapFiles:
- %currentWorkingDirectory%/tests/static-analysis-stubs/wordpress-defines.stub
Copy link
Contributor

@szepeviktor szepeviktor Jul 13, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

- %currentWorkingDirectory%/vendor/php-stubs/wordpress-stubs/wordpress-stubs.php is necessary if you don't use my package

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@westonruter Now we are here :)

@westonruter westonruter merged commit ed653ce into develop Jul 14, 2021
@westonruter westonruter deleted the improve/phpstan branch July 14, 2021 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants