Skip to content

Commit

Permalink
Update PhpCodeSniffer and composer.json
Browse files Browse the repository at this point in the history
  • Loading branch information
mr-chetan committed Jan 10, 2024
1 parent 3140ed4 commit ea25f22
Show file tree
Hide file tree
Showing 4 changed files with 72 additions and 1 deletion.
4 changes: 4 additions & 0 deletions app/Support/PhpCodeSniffer.php
Original file line number Diff line number Diff line change
Expand Up @@ -83,13 +83,17 @@ private function installDevanoxCodingStandard(): void
}

/**
* Config uses a private static property $overriddenDefaults
* which does't allow us to update the config between runs
* we need to reset it so we can also lint in the fix command.
*/
private function resetConfig(string $tool): void
{
if (defined('PHP_CODESNIFFER_CBF') === false) {
define('PHP_CODESNIFFER_CBF', $tool === 'runPHPCBF');
}

invade(new Config)->overriddenDefaults = [];

Check failure on line 96 in app/Support/PhpCodeSniffer.php

View workflow job for this annotation

GitHub Actions / PHP 8.2 - ubuntu-22.04

Access to private property Spatie\Invade\Invader<PHP_CodeSniffer\Config>::$overriddenDefaults.
}

private function getConfigFile(): string
Expand Down
Binary file modified builds/revive
Binary file not shown.
1 change: 1 addition & 0 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@
"nunomaduro/termwind": "v1.15.1",
"pestphp/pest": "^v2.30",
"rector/rector": "^0.19.0",
"spatie/invade": "^1.1.1",
"tightenco/tlint": "^9.2"
},
"autoload": {
Expand Down
68 changes: 67 additions & 1 deletion composer.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit ea25f22

Please sign in to comment.