Skip to content

ci: fetch tags on checkout - #1053

Open
j-zimnowoda wants to merge 3 commits into
mainfrom
APL-2053-2
Open

ci: fetch tags on checkout#1053
j-zimnowoda wants to merge 3 commits into
mainfrom
APL-2053-2

Conversation

@j-zimnowoda

@j-zimnowoda j-zimnowoda commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

This is required to fetch all tags so that release-get-tag can find the highest tag on the branch.

Copilot AI lite review requested due to automatic review settings August 7, 2026 11:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 release GitHub Actions workflow to ensure git tags are available during checkout so the release automation can compute and create the correct release tag.

Changes:

  • Fetch tags during the release-get-tag job checkout to support tag-based release tag calculation.
  • Adjust checkout settings in release-software to use a shallow fetch plus tag fetching.
Suppressed comments (1)

.github/workflows/release-software.yml:66

  • This job previously used fetch-depth: 0, which is the same approach used elsewhere in the repo for release/tag derivation (see .github/workflows/cut-release-branch.yml:39-40). Switching to a shallow checkout (fetch-depth: 1) can break steps/actions that need to compare against older tags/commits (even if tag refs are present), and it’s inconsistent with the established workflow pattern. Consider keeping a full fetch here.
          ref: ${{ env.RELEASE_BRANCH }}
          fetch-depth: 1
          fetch-tags: true

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

Comment on lines 37 to +39
fetch-depth: 1

# This is required to fetch all tags so that release-get-tag can find the highest tag on the branch
fetch-tags: true
Copilot AI review requested due to automatic review settings August 7, 2026 12:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings August 10, 2026 10:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

3 participants