Skip to content

Commit

Permalink
Fix psalm
Browse files Browse the repository at this point in the history
  • Loading branch information
vjik committed Feb 7, 2024
1 parent 4124294 commit 4209ab7
Show file tree
Hide file tree
Showing 6 changed files with 34 additions and 4 deletions.
11 changes: 10 additions & 1 deletion .github/workflows/static.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ on:
- 'phpunit.xml.dist'

push:
branches: [ 'master' ]
paths-ignore:
- 'docs/**'
- 'README.md'
Expand All @@ -28,4 +29,12 @@ jobs:
os: >-
['ubuntu-latest']
php: >-
['7.4', '8.0', '8.1']
['8.1', '8.2', '8.3']
psalm74-80:
uses: yiisoft/actions/.github/workflows/psalm.yml@master
with:
psalm-config: psalm74-80.xml
os: >-
['ubuntu-latest']
php: >-
['7.4', '8.0']
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@
"rector/rector": "^1.0.0",
"roave/infection-static-analysis-plugin": "^1.16",
"spatie/phpunit-watcher": "^1.23",
"vimeo/psalm": "^4.30|^5.6",
"vimeo/psalm": "^4.30|^5.21",
"yiisoft/di": "^1.1"
},
"autoload": {
Expand Down
4 changes: 4 additions & 0 deletions psalm.xml
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,8 @@
<directory name="vendor" />
</ignoreFiles>
</projectFiles>
<issueHandlers>
<MixedAssignment errorLevel="suppress" />
<RiskyTruthyFalsyComparison errorLevel="suppress" />
</issueHandlers>
</psalm>
19 changes: 19 additions & 0 deletions psalm74-80.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
<?xml version="1.0"?>
<psalm
errorLevel="1"
findUnusedBaselineEntry="true"
findUnusedCode="false"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xmlns="https://getpsalm.org/schema/config"
xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd"
>
<projectFiles>
<directory name="src" />
<ignoreFiles>
<directory name="vendor" />
</ignoreFiles>
</projectFiles>
<issueHandlers>
<MixedAssignment errorLevel="suppress" />
</issueHandlers>
</psalm>
1 change: 0 additions & 1 deletion src/CsrfMiddleware.php
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,6 @@ private function getTokenFromRequest(ServerRequestInterface $request): ?string
{
$parsedBody = $request->getParsedBody();

/** @var mixed $token */
$token = $parsedBody[$this->parameterName] ?? null;
if (empty($token)) {
$headers = $request->getHeader($this->headerName);
Expand Down
1 change: 0 additions & 1 deletion src/Synchronizer/Storage/SessionCsrfTokenStorage.php
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ public function __construct(SessionInterface $session, string $key = self::KEY)

public function get(): ?string
{
/** @var mixed $value */
$value = $this->session->get($this->key);
return is_string($value) ? $value : null;
}
Expand Down

0 comments on commit 4209ab7

Please sign in to comment.