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

Speedup psalm test with only diff #155

Open
menno-ll opened this issue Jun 11, 2019 · 0 comments
Open

Speedup psalm test with only diff #155

menno-ll opened this issue Jun 11, 2019 · 0 comments

Comments

@menno-ll
Copy link
Member

In the composer test script, testing with psalm takes a long time. We can make it faster for example if we only check changed files. maybe we should consider that. See command options with /vendor/bin/psalm --help or https://github.com/vimeo/psalm/blob/master/docs/running_psalm/command_line_usage.md

Command outputs:

➜  Clarkson-Core git:(feature/psalm) vendor/bin/psalm --threads=8 --diff --diff-methods --show-info=false
Scanning files...
Analyzing files...

------------------------------
No errors found!
------------------------------

Checks took **7.10** seconds and used 313.776MB of memory
No files analyzed
➜  Clarkson-Core git:(feature/psalm) vendor/bin/psalm --show-info=false                                  
Scanning files...
Analyzing files...

------------------------------
No errors found!
------------------------------

Checks took **13.76** seconds and used 361.078MB of memory
Psalm was able to infer types for 85.2657% of the codebase
➜  Clarkson-Core git:(feature/psalm) 

Originally posted by @menno-ll in #151 (comment)

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

No branches or pull requests

2 participants