Skip to content

chore: Move comments event handler to use proper event dispatcher #44060

chore: Move comments event handler to use proper event dispatcher

chore: Move comments event handler to use proper event dispatcher #44060

Triggered via pull request June 18, 2024 12:11
Status Cancelled
Total duration 16m 10s
Artifacts

static-code-analysis.yml

on: pull_request
static-code-analysis
4m 24s
static-code-analysis
static-code-analysis-security
5m 49s
static-code-analysis-security
static-code-analysis-ocp
2m 53s
static-code-analysis-ocp
Fit to window
Zoom out
Zoom in

Annotations

8 errors
InvalidDocblock: lib/public/Comments/ICommentsEventHandler.php#L14
lib/public/Comments/ICommentsEventHandler.php:14:1: InvalidDocblock: Typo in @deprecated for classes/interfaces in OCP. (see https://psalm.dev/008)
InvalidDocblock: lib/public/Comments/ICommentsEventHandler.php#L20
lib/public/Comments/ICommentsEventHandler.php:20:2: InvalidDocblock: Typo in @deprecated for method in OCP. (see https://psalm.dev/008)
static-code-analysis-ocp
Process completed with exit code 2.
MissingTemplateParam: apps/comments/lib/Listener/CommentsEventListener.php#L35
apps/comments/lib/Listener/CommentsEventListener.php:35:40: MissingTemplateParam: OCA\Comments\Listener\CommentsEventListener has missing template params when extending OCP\EventDispatcher\IEventListener, expecting 1 (see https://psalm.dev/182)
static-code-analysis
Process completed with exit code 2.
static-code-analysis-security
Canceling since a higher priority waiting request for 'static-code-analysis-chore/comments-event-legacy' exists
static-code-analysis-security
The operation was canceled.
static-code-analysis-security
Path does not exist: results.sarif