Skip to content

chore(CI): Adjust testing matrix for Nextcloud 32 on main#11623

Merged
ChristophWurst merged 1 commit intomainfrom
update-main-target-versions
Sep 5, 2025
Merged

chore(CI): Adjust testing matrix for Nextcloud 32 on main#11623
ChristophWurst merged 1 commit intomainfrom
update-main-target-versions

Conversation

@nickvergessen
Copy link
Member

Signed-off-by: Joas Schilling coding@schilljs.com

@kesselb
Copy link
Contributor

kesselb commented Sep 5, 2025

Run into the same psalm failure in #11628 (comment). What are we going to do? ;)

@ChristophWurst
Copy link
Member

Run into the same psalm failure in #11628 (comment). What are we going to do? ;)

Annotate the more general type 🤞

@ChristophWurst
Copy link
Member

That made it worse 🙈

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst force-pushed the update-main-target-versions branch from 3bc6b16 to f61ba25 Compare September 5, 2025 14:05
@ChristophWurst
Copy link
Member

I'm taking a cheap shortcut and won't add stable30 to the Psalm matrix now because

  1. It worked so far, so the type error is more of a theoretical concern than a practical issue
  2. We will drop stable30 from the app in 1-2 weeks as it is EOLing

@ChristophWurst ChristophWurst merged commit f2c883a into main Sep 5, 2025
42 checks passed
@ChristophWurst ChristophWurst deleted the update-main-target-versions branch September 5, 2025 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants