Skip to content

dev - #200

Merged
RambokDev merged 98 commits into
mainfrom
dev
Mar 6, 2026
Merged

dev#200
RambokDev merged 98 commits into
mainfrom
dev

Conversation

@RambokDev

@RambokDev RambokDev commented Mar 6, 2026 •

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Chores
    • Improved Docker image tagging strategy with per-output tag generation (VERSION_TAG, MINOR_TAG, MAJOR_TAG, LATEST_TAG).
    • Enhanced release notifications to use finalized version information instead of draft tags.

charlesgauthereau added 30 commits March 6, 2026 09:10
# Conflicts:
#	.github/workflows/auto-release.yml
# Conflicts:
#	.github/workflows/auto-release.yml
# Conflicts:
#	.github/workflows/auto-release.yml
# Conflicts:
#	.github/workflows/auto-release.yml
# Conflicts:
#	.github/workflows/auto-release.yml
# Conflicts:
#	.github/workflows/auto-release.yml
# Conflicts:
#	.github/workflows/auto-release.yml
# Conflicts:
#	.github/workflows/auto-release.yml
# Conflicts:
#	.github/workflows/auto-release.yml
# Conflicts:
#	.github/workflows/auto-release.yml
@coderabbitai

coderabbitai Bot commented Mar 6, 2026 •

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 1046ff02-7995-4e31-8ef9-183031bb4f49

📥 Commits

Reviewing files that changed from the base of the PR and between 0126a82 and 0988fa0.

📒 Files selected for processing (2)
  • .github/workflows/docker.yml
  • .github/workflows/release.yml

📝 Walkthrough

Walkthrough

GitHub Actions workflows refactored to restructure Docker image tagging. The docker.yml workflow now outputs tags individually (VERSION_TAG, MINOR_TAG, MAJOR_TAG, LATEST_TAG) rather than a combined string, while release.yml updates the Discord notification to use the finalized version output instead of draft tag.

Changes

Cohort / File(s) Summary
Docker Tagging Output Restructure
.github/workflows/docker.yml
Replaces single TAGS aggregation with per-output tagging strategy. Emits individual outputs (VERSION_TAG, MINOR_TAG, MAJOR_TAG, LATEST_TAG) and updates docker/metadata-action to consume per-tag values instead of combined tags string.
Discord Notification Parameter Update
.github/workflows/release.yml
Changes notify-discord step's release_tag parameter from draft_tag to version output, switching notification to use finalized version instead of draft tag.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • dev #175 — Modifies identical workflow files to restructure tag production and Docker publish configuration
  • dev #163 — Adjusts GitHub Actions release/tagging behavior and downstream step consumption
  • dev #177 — Updates release workflow's tag handling in notify-discord step

Poem

🐰 The tags, they scattered far and wide,
Per-output dancing, side by side,
No more TAGS strung in twisted thread,
VERSION, MINOR, MAJOR spread—
And Discord learns the version's song,
No draft confusion clouds along! 🎉

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud

sonarqubecloud Bot commented Mar 6, 2026

Copy link
Copy Markdown

@RambokDev
RambokDev merged commit 6593a8e into main Mar 6, 2026
3 of 4 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request Mar 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant