Skip to content

chore(ci): name the actual tag in the version-matrix pin comments - #2129

Merged
AndyScherzinger merged 1 commit into
mainfrom
chore/noid/fix-version-matrix-tag-comments
Nov 11, 2026
Merged

AndyScherzinger merged 1 commit into
mainfrom
chore/noid/fix-version-matrix-tag-comments

Conversation

@AndyScherzinger

Copy link
Copy Markdown
Member

Follow-up to #1967, where @Croydon pointed out in #1967 (comment) that two different commits cannot both be tag v1.0.0.

All six workflows pin icewind1991/nextcloud-version-matrix to cd0211ffcef1065e2020cd579e4843b8746e7a58, which is tag v1.3.3. The comments next to it disagree: four read v1.3.3.3.1.3.3, two read v1.0.0. For reference, v1.0.0 is c2bf575a3516752db5ce2915499d3f694885e2c7 and is not used anywhere in this repository.

workflow comment before
appstore-build-publish.yml # v1.3.3.3.1.3.3
lint-php-cs.yml # v1.3.3.3.1.3.3
lint-php.yml # v1.0.0
phpunit-unit.yml # v1.0.0
psalm-matrix.yml # v1.3.3.3.1.3.3
update-nextcloud-ocp-matrix.yml # v1.3.3.3.1.3.3

All six now read # v1.3.3. Comments only — the pinned commit is unchanged, so no job behaviour changes.

Not bumping to the current v1.3.4 (71959de73c244b11ecbf4503b87c41d65b0f64ae) here, so that this stays a pure comment fix; happy to do that separately if wanted.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

All six workflows pin icewind1991/nextcloud-version-matrix to
cd0211ffcef1065e2020cd579e4843b8746e7a58, which is tag v1.3.3. Four of
the comments read v1.3.3.3.1.3.3, two read v1.0.0. v1.0.0 is
c2bf575a3516752db5ce2915499d3f694885e2c7 and is not used here.

Comments only; the pinned commit is unchanged.

Assisted-by: Claude Code:claude-opus-5[1m]
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@nextcloud-command nextcloud-command added the AI assisted This PR contains AI-assisted commits label Nov 11, 2026
@AndyScherzinger AndyScherzinger mentioned this pull request Nov 11, 2026
1 task done
@codecov

codecov Bot commented Nov 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@AndyScherzinger
AndyScherzinger merged commit a96c873 into main Nov 11, 2026
40 checks passed
@AndyScherzinger
AndyScherzinger deleted the chore/noid/fix-version-matrix-tag-comments branch November 11, 2026 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI assisted This PR contains AI-assisted commits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants