Skip to content

Conversation

@nextcloud-command
Copy link
Contributor

Auto-generated update of nextcloud/ocp dependency

@nextcloud-command nextcloud-command added 3. to review dependencies Pull requests that update a dependency file labels Jan 5, 2025
@nextcloud-command nextcloud-command force-pushed the automated/noid/main-update-nextcloud-ocp branch 2 times, most recently from eae1abb to 1f6e6ff Compare January 19, 2025 02:56
@nextcloud-command nextcloud-command force-pushed the automated/noid/main-update-nextcloud-ocp branch from 1f6e6ff to 426f4b7 Compare January 26, 2025 02:52
@enjeck enjeck force-pushed the automated/noid/main-update-nextcloud-ocp branch from 426f4b7 to 990f842 Compare January 30, 2025 04:23
Copy link
Contributor

@enjeck enjeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and CI tests now pass

@enjeck
Copy link
Contributor

enjeck commented Jan 30, 2025

@JuliaKirschenheuter Most of the changes are autogenerated, but I added a few commits to fix the CI errors. Would appreciate a review of my changes before we merge

nextcloud-command and others added 5 commits February 1, 2025 07:10
Signed-off-by: Cleopatra Enjeck M. <[email protected]>
This fixes the error message: ERROR: ArgumentTypeCoercion - vendor/nextcloud/ocp/OCP/AppFramework/Controller.php:56:7 - Argument 1 of OCP\AppFramework\Http\JSONResponse::__construct expects JsonSerializable|array<array-key, mixed>|null|scalar|stdClass, but parent type array<array-key, mixed>|null|object|scalar provided (see https://psalm.dev/193)

Signed-off-by: Cleopatra Enjeck M. <[email protected]>
@enjeck enjeck force-pushed the automated/noid/main-update-nextcloud-ocp branch from c542ffa to bbcd9e3 Compare February 1, 2025 07:10
@enjeck enjeck merged commit 9e113ec into main Feb 1, 2025
30 checks passed
@enjeck enjeck deleted the automated/noid/main-update-nextcloud-ocp branch February 1, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants