Skip to content

Commit

Permalink
Merge pull request #47 from patchlevel/allow-psalm-v6
Browse files Browse the repository at this point in the history
Allow psalm 6 as dep, update workflow to support both versions
  • Loading branch information
DanielBadura authored Feb 9, 2025
2 parents e6ad135 + 90fa398 commit c6896f1
Show file tree
Hide file tree
Showing 5 changed files with 1,595 additions and 337 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/psalm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ jobs:
strategy:
matrix:
dependencies:
- "lowest"
- "locked"
php-version:
- "8.3"
Expand All @@ -41,4 +42,9 @@ jobs:
dependency-versions: ${{ matrix.dependencies }}

- name: "psalm"
if: ${{ matrix.dependencies != 'locked' }}
run: "vendor/bin/psalm"

- name: "psalm"
if: ${{ matrix.dependencies == 'locked' }}
run: "vendor/bin/psalm --shepherd --stats"
29 changes: 28 additions & 1 deletion baseline.xml
Original file line number Diff line number Diff line change
@@ -1,2 +1,29 @@
<?xml version="1.0" encoding="UTF-8"?>
<files psalm-version="5.23.1@8471a896ccea3526b26d082f4461eeea467f10a4"/>
<files psalm-version="6.4.0@04f312ac6ea48ba1c3e5db4d815bf6d74641c0ee">
<file src="src/Plugin.php">
<UnusedClass>
<code><![CDATA[Plugin]]></code>
</UnusedClass>
</file>
<file src="tests/Valid/NameChanged.php">
<PossiblyUnusedMethod>
<code><![CDATA[__construct]]></code>
</PossiblyUnusedMethod>
</file>
<file src="tests/Valid/Profile.php">
<PossiblyUnusedMethod>
<code><![CDATA[applyProfileCreated]]></code>
<code><![CDATA[create]]></code>
<code><![CDATA[id]]></code>
<code><![CDATA[name]]></code>
</PossiblyUnusedMethod>
</file>
<file src="tests/Valid/ProfileProjection.php">
<PossiblyUnusedParam>
<code><![CDATA[$id]]></code>
</PossiblyUnusedParam>
<UnusedClass>
<code><![CDATA[ProfileProjection]]></code>
</UnusedClass>
</file>
</files>
5 changes: 4 additions & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
],
"require": {
"php": "~8.1 || ~8.2 || ~8.3",
"vimeo/psalm": "^5.0.0",
"vimeo/psalm": "^5.14.0 || ^6.0.0",
"ext-simplexml": "*"
},
"require-dev": {
Expand All @@ -29,6 +29,9 @@
"roave/security-advisories": "dev-master",
"symfony/var-dumper": "^7.0.0"
},
"conflict": {
"netresearch/jsonmapper": "^1.0"
},
"config": {
"preferred-install": {
"*": "dist"
Expand Down
Loading

0 comments on commit c6896f1

Please sign in to comment.