Repository navigation
fix(release): fix the release script to find the correct last tag - #3319
pratap0007 wants to merge 1 commit into
Conversation
tekton-robot
added
the
release-note-none
tekton-robot
added
the
size/M
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The fallback can select an unrelated release series or reuse the target tag, producing incorrect changelogs.
Review effort: Balanced
Findings: 2
Open (2)
What changed in this PR
Updates the release script to select changelog baselines for minor and patch releases.
Changes:
- Looks for a same-series tag, with a latest-overall fallback.
- Determines release type from the patch version rather than branch existence.
- Uses the selected tag when generating the changelog.
| File | Description |
|---|---|
| tekton/release.sh | Updates tag selection, release classification, and release messaging. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
For minor release lasttag would be latest tag For patch release lasttag would be respective branch latest tag Signed-off-by: pratap0007 <shverma@redhat.com>
d15c71d to
01d553d
Compare
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: divyansh42 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
tekton-robot
added
the
approved
|
/retest |
2 similar comments
|
/retest |
|
/retest |
fix(release): fix the release script to find the correct last tag
Changes
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:
make checkmake generatedSee the contribution guide
for more details.
Release Notes