Skip to content

ci: Exclude the nightly tags when filtering unseen commits#179

Open
lens0021 wants to merge 2 commits intoamber-lang:mainfrom
lens0021:exclude-nightly
Open

ci: Exclude the nightly tags when filtering unseen commits#179
lens0021 wants to merge 2 commits intoamber-lang:mainfrom
lens0021:exclude-nightly

Conversation

@lens0021
Copy link
Contributor

@lens0021 lens0021 commented Feb 10, 2026

The workflow tell us which commits are missing in the documentations. Currently, it is not so useful because nightly and staging are too close.

image

Summary by CodeRabbit

  • Chores
    • Improved release tag identification by using version-aware sorting and excluding nightly tags so workflows select the correct latest release tag for computing commit ranges.

@lens0021 lens0021 self-assigned this Feb 10, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 10, 2026

📝 Walkthrough

Walkthrough

The prerelease workflow now selects the latest tag using a version-aware sort and filters out nightly tags (git tag --sort=version:refname | grep -v nightly | tail --lines=1), changing which tag defines the commit range for prerelease subjects.

Changes

Cohort / File(s) Summary
Workflow Tag Selection
\.github/workflows/prerelease.yml
Replaced simple `git tag

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped through tags both old and sprightly,
Chose versions true and skipped the nightly,
Clean commits now guide the way,
A stable path for release day! 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: excluding nightly tags from the commit filtering logic in the CI workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉


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

@vercel
Copy link

vercel bot commented Feb 10, 2026

@lens0021 is attempting to deploy a commit to the Marble Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In @.github/workflows/prerelease.yml:
- Line 26: The current LATEST_TAG assignment uses plain `git tag`, which sorts
lexicographically; update the command that sets LATEST_TAG to use version-aware
sorting by adding `--sort=version:refname` so the pipeline picks the latest
semantic version (e.g., replace the `git tag | grep -v nightly | tail` pipeline
referenced by LATEST_TAG with a `git tag --sort=version:refname | grep -v
nightly | tail -n1` variant).

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
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.

2 participants