feat: policy signature flow #10051
vitormattospsalm.yml
on: pull_request
static-psalm-analysis
51s
Annotations
5 errors and 1 warning
|
static-psalm-analysis
Process completed with exit code 2.
|
|
LessSpecificReturnStatement:
lib/Controller/PolicyController.php#L118
lib/Controller/PolicyController.php:118:10: LessSpecificReturnStatement: The type 'OCP\AppFramework\Http\DataResponse<200, array{policy: array{allowChildOverride: bool, allowedValues: list<null|scalar>, policyKey: string, scope: 'global'|'system', value: array<array-key, mixed>|null|scalar, visibleToChild: bool}}, array<never, never>>' is more general than the declared return type 'OCP\AppFramework\Http\DataResponse<200, array{policy: array{allowChildOverride: bool, allowedValues: list<null|scalar>, policyKey: string, scope: 'global'|'system', value: null|scalar, visibleToChild: bool}}, array<never, never>>' for OCA\Libresign\Controller\PolicyController::getSystem (see https://psalm.dev/129)
|
|
MoreSpecificReturnType:
lib/Controller/PolicyController.php#L91
lib/Controller/PolicyController.php:91:13: MoreSpecificReturnType: The declared return type 'OCP\AppFramework\Http\DataResponse<200, array{policy: array{allowChildOverride: bool, allowedValues: list<null|scalar>, policyKey: string, scope: 'global'|'system', value: null|scalar, visibleToChild: bool}}, array<never, never>>' for OCA\Libresign\Controller\PolicyController::getSystem is more specific than the inferred return type 'OCP\AppFramework\Http\DataResponse<200, array{policy: array{allowChildOverride: bool, allowedValues: list<null|scalar>, policyKey: string, scope: 'global'|'system', value: array<array-key, mixed>|null|scalar, visibleToChild: bool}}, array<never, never>>' (see https://psalm.dev/070)
|
|
InvalidReturnStatement:
lib/Controller/AdminController.php#L858
lib/Controller/AdminController.php:858:10: InvalidReturnStatement: The inferred type 'OCP\AppFramework\Http\DataResponse<200, array{isDefault: bool, preview_height: int, preview_width: int, preview_zoom: int, template: string}, array<never, never>>' does not match the declared return type 'OCP\AppFramework\Http\DataResponse<200, array{isDefault: bool, preview_height: int, preview_width: int, template: string}, array<never, never>>' for OCA\Libresign\Controller\AdminController::getFooterTemplate (see https://psalm.dev/128)
|
|
InvalidReturnType:
lib/Controller/AdminController.php#L850
lib/Controller/AdminController.php:850:13: InvalidReturnType: The declared return type 'OCP\AppFramework\Http\DataResponse<200, array{isDefault: bool, preview_height: int, preview_width: int, template: string}, array<never, never>>' for OCA\Libresign\Controller\AdminController::getFooterTemplate is incorrect, got 'OCP\AppFramework\Http\DataResponse<200, array{isDefault: bool, preview_height: int, preview_width: int, preview_zoom: int, template: string}, array<never, never>>' (see https://psalm.dev/011)
|
|
static-psalm-analysis
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: icewind1991/nextcloud-version-matrix@8a7bac6300b2f0f3100088b297995a229558ddba. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|