Skip to content

Conversation

@aholstrup1
Copy link
Collaborator

❔What, Why & How

Related to issue: #2094

When AL-Go tries to download the latest release it will look in a specific branch. Currently, we use whatever is contained in GITHUB_REF_NAME as the branch but this doesn't always work for pull requests. If you use the "pull_request" trigger then GITHUB_REF_NAME will be the name of the merge branch when it should be the name of the base branch.

Suggested fix: Just use the base branch whenever possible (https://docs.github.com/en/actions/reference/workflows-and-actions/contexts). If it's not available, use the ref_name.

✅ Checklist

  • Add tests (E2E, unit tests)
  • Update RELEASENOTES.md
  • Update documentation (e.g. for new settings or scenarios)
  • Add telemetry

@aholstrup1 aholstrup1 requested a review from a team as a code owner January 23, 2026 12:30
Copilot AI review requested due to automatic review settings January 23, 2026 12:30
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes incorrect “previous release” selection in PR builds by using the PR base branch (when available) instead of the PR merge ref name, aligning release lookup with the intended target branch (Issue #2094).

Changes:

  • Use GITHUB_BASE_REF as the release-lookup branch when present; otherwise fall back to GITHUB_REF_NAME.
  • Add release note entry for Issue #2094.

Reviewed changes

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

File Description
RELEASENOTES.md Documents the fix for Issue #2094 in the release notes.
Actions/RunPipeline/RunPipeline.ps1 Adjusts previous-release lookup to prefer the PR base branch context.

💡 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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants