Skip to content

fix(release): improve previous tag detection and use stable goreleaser image - #3070

Open
pratap0007 wants to merge 1 commit into
tektoncd:mainfrom
pratap0007:update-rel-script
Open

fix(release): improve previous tag detection and use stable goreleaser image#3070
pratap0007 wants to merge 1 commit into
tektoncd:mainfrom
pratap0007:update-rel-script

Conversation

@pratap0007

Copy link
Copy Markdown
Contributor

fix(release): improve previous tag detection and use stable goreleaser image

Restructure release.sh to detect patch vs minor release before resolving
the previous tag, so patch releases correctly scope tag lookup to the
same minor version. Simplify changelog commit range to use git rev range
(prev_tag..HEAD) instead of date-based --since filtering.
Pin goreleaser to stable v2.17.1, replacing the nightly build.

Changes

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

  • Includes tests (if functionality changed/added)
  • Run the code checkers with make check
  • Regenerate the manpages, docs and go formatting with make generated
  • Commit messages follow commit message best practices

See the contribution guide
for more details.

Release Notes

…r image

  Restructure release.sh to detect patch vs minor release before resolving
  the previous tag, so patch releases correctly scope tag lookup to the
  same minor version. Simplify changelog commit range to use git rev range
  (prev_tag..HEAD) instead of date-based --since filtering. Pin goreleaser
  to stable v2.17.1, replacing the nightly build.

Signed-off-by: Shiv Verma <shverma@redhat.com>
@tekton-robot tekton-robot added the release-note Denotes a PR that will be considered when it comes time to generate release notes. label Jul 28, 2026
@tekton-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please ask for approval from pratap0007 after the PR has been reviewed.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jul 28, 2026
@divyansh42
divyansh42 requested a review from Copilot July 28, 2026 14:31

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Tekton CLI release automation to more reliably determine the correct “previous tag” for changelog generation (especially for patch releases) and to use a pinned, stable GoReleaser image in the Tekton release pipeline.

Changes:

  • Detect patch vs. minor release earlier, and scope patch-release previous-tag lookup to the same vX.Y.* series.
  • Simplify changelog commit selection to a straightforward Git revision range (${prev_tag}..HEAD) instead of date-based filtering.
  • Pin the release pipeline’s GoReleaser image to goreleaser/goreleaser:v2.17.1 (stable), replacing the nightly image.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
tekton/release.sh Reworks previous-tag detection and switches changelog commit selection to ${prev_tag}..HEAD.
tekton/release-pipeline.yml Pins the GoReleaser task image to the stable v2.17.1 container.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note Denotes a PR that will be considered when it comes time to generate release notes. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants