Skip to content

ci(fleet): centralize trunk config and gate release tags#39

Merged
JSONbored merged 4 commits intomainfrom
codex/fleet-workflow-drift-fixes
Apr 23, 2026
Merged

ci(fleet): centralize trunk config and gate release tags#39
JSONbored merged 4 commits intomainfrom
codex/fleet-workflow-drift-fixes

Conversation

@JSONbored
Copy link
Copy Markdown
Owner

@JSONbored JSONbored commented Apr 23, 2026

Summary

  • consume the shared unraid-aio Trunk plugin config
  • only emit immutable release tags on real release commits
  • document the publish and release tag behavior

Validation

  • trunk check --show-existing --all
  • python3 scripts/validate-template.py
  • pytest tests/unit tests/template
  • pytest tests/integration -m integration
  • ./trunk-analytics-cli validate --junit-paths "reports/pytest-unit.xml,reports/pytest-integration.xml"
  • git diff --check

@JSONbored JSONbored changed the title ci(release): remove automation flag and align publish flow ci(release): standardize fleet publish and validation flow Apr 23, 2026
@JSONbored JSONbored self-assigned this Apr 23, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5659c99272

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

;;
*)
release_package_tag="${upstream_version}-aio.1"
if printf '%s\n' "${commit_message}" | grep -Fxq "chore(release): ${changelog_version}"; then
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Accept release commit titles with PR suffix

The release tag detection now requires an exact line match on chore(release): <version>, but GitHub squash merges commonly produce chore(release): <version> (#123). In that case release_package_tag stays empty, so the publish job skips pushing the immutable <upstream>-aio.N image tag and related wrapper metadata on the release commit. This breaks the documented release flow for repos that use the default squash-merge title format.

Useful? React with 👍 / 👎.

@JSONbored JSONbored changed the title ci(release): standardize fleet publish and validation flow ci(fleet): centralize trunk config and gate release tags Apr 23, 2026
@JSONbored JSONbored merged commit e841d33 into main Apr 23, 2026
12 checks passed
@JSONbored JSONbored deleted the codex/fleet-workflow-drift-fixes branch April 23, 2026 09:50
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