From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/ci: disable mr-label-maker-test Date: Mon, 25 May 2026 19:21:31 +1000 Message-ID: In-Reply-To: <20260522-drm-ci-mr-label-v1-1-39f778123b30@oss.qualcomm.com> References: <20260522-drm-ci-mr-label-v1-1-39f778123b30@oss.qualcomm.com> <20260522-drm-ci-mr-label-v1-1-39f778123b30@oss.qualcomm.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Correctness:** The patch adds a job override block that matches the estab= lished pattern in the file: ```yaml mr-label-maker-test: rules: - when: never ``` This is identical in structure to the surrounding disabled jobs (`deploy-do= cs` at line 381, `linkcheck-docs` at line 385, `test-docs` at line 393). Th= e placement is alphabetically/logically correct =E2=80=94 inserted between = `linkcheck-docs` and `test-docs`. **Indentation:** Uses the same 3-space indent for `rules:` and 4-space inde= nt for `- when: never`, consistent with the rest of the file. **No concerns:** - The commit message adequately explains both the problem (job got enabled,= blocks pipeline) and the rationale (MR labelling is not used for DRM CI). - The `Link:` tag to the failing pipeline is helpful for reviewers to verif= y the issue. - The `Signed-off-by` is present. - The `base-commit` is specified, making it easy to apply. **Reviewed-by worthy.** This is a clean, minimal fix following existing con= ventions. --- Generated by Claude Code Patch Reviewer