Skip to content

chore: Update psalm to v6.16.0 - #13525

Draft
kesselb wants to merge 1 commit into
mainfrom
vendor-bin-psalm
Draft

chore: Update psalm to v6.16.0#13525
kesselb wants to merge 1 commit into
mainfrom
vendor-bin-psalm

Conversation

@kesselb

@kesselb kesselb commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Refined version of #11224

Psalm works on CI, but is broken locally with newer PHP versions.

Main different to the previous attempt: psalm/phar is moved to vendor-bin and merged with psalm-stubs. composer.json forces php to 8.1 (for nextcloud 32). For psalm itself that is not necessary (and covered by the php version in psalm.xml). Moving the vendor-bin allows to use the latest version.

Using vimeo/psalm (instead of the phar) failed with Fatal error: Cannot redeclare function Amp\delay() (previously declared in vendor/amphp/amp/lib/functions.php:131) in vendor-bin/psalm/vendor/amphp/amp/src/functions.php on line 64, hence we stick with phar for now.

馃 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@nextcloud-command nextcloud-command added the AI assisted This PR contains AI-assisted commits label Aug 19, 2026
@kesselb
kesselb marked this pull request as draft August 19, 2026 21:41
@kesselb

kesselb commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author
Problem 1
    - psalm/phar is locked to version 6.16.1 and an update of this package was not requested.
    - psalm/phar 6.16.1 requires php ^8.2 -> your php version (8.1.34) does not satisfy that requirement.

I'd forgot 馃檲

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

Labels

AI assisted This PR contains AI-assisted commits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants