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.

Sign up for GitHub

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

Jump to bottom

Bump PHPStan to level 2 when doing deprecations #267

Merged
merged 4 commits into from
Apr 29, 2022
Merged

Bump PHPStan to level 2 when doing deprecations #267

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a07bfb6
Bump PHPStan to level 2 when doing deprecations
mglaman Mar 3, 2022
4a19447
update circleci to allow exit code 1 failures but not 255
mglaman Mar 3, 2022
94ba63c
always ignore `Unsafe usage of new static()`
mglaman Apr 29, 2022
57680ca
Update README
mglaman Apr 29, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Bump PHPStan to level 2 when doing deprecations
  • Loading branch information
@mglaman
mglaman committed Apr 29, 2022
commit a07bfb641a185dc7de0b3c661e2b12ecd338e995
4 changes: 2 additions & 2 deletions src/Command/CheckCommand.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -156,14 +156,14 @@ protected function execute(InputInterface $input, OutputInterface $output): int
}

if ($this->isAnalysisCheck) {
$configuration_data['parameters']['level'] = 4;
$configuration_data['parameters']['level'] = 6;

$ignored_analysis_errors = [
'#Unsafe usage of new static\(\)#'
];
$configuration_data['parameters']['ignoreErrors'] = array_merge($ignored_analysis_errors, $configuration_data['parameters']['ignoreErrors']);
} else {
$configuration_data['parameters']['customRulesetUsed'] = true;
$configuration_data['parameters']['level'] = 2;
}

if ($this->isDeprecationsCheck) {
Expand Down