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

Fix Psalm issues modules/ #1046

Open
wants to merge 30 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@tvdijen
Copy link
Member

tvdijen commented Feb 4, 2019

I ran a check with the latest version of Psalm, which kinda opened Pandora's box on me..
This PR attempts to fix as much of the low-hanging fruit as possible for files in the modules-directory.

Left for the modules-dir:
415 errors found
100 other issues found

@tvdijen tvdijen added this to the 1.18 milestone Feb 4, 2019

@tvdijen tvdijen force-pushed the psalm-modules branch 2 times, most recently from 32183db to 9b0b229 Feb 11, 2019

@tvdijen tvdijen force-pushed the psalm-modules branch from 9b0b229 to 08c3f1b Feb 11, 2019

@tvdijen tvdijen force-pushed the psalm-modules branch from 08c3f1b to 9ec8617 Feb 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment