Skip to content

Get backport workflow to work in NuGet repos#6857

Merged
zivkan merged 1 commit intodevfrom
dev-zivkan-fix-backport-bot
Oct 15, 2025
Merged

Get backport workflow to work in NuGet repos#6857
zivkan merged 1 commit intodevfrom
dev-zivkan-fix-backport-bot

Conversation

@zivkan
Copy link
Copy Markdown
Member

@zivkan zivkan commented Oct 15, 2025

Bug

Fixes: infrastructure

Description

The workflow has a condition: https://github.com/dotnet/arcade/blob/5c32309ec8a87f89f64e5fe1e56c60637ac057f9/.github/workflows/backport-base.yml#L30

It looks like we need to pass in NuGet as the repository_owners to allow the condition to be true.

PR Checklist

  • Meaningful title, helpful description and a linked NuGet/Home issue
  • Added tests N/A
  • Link to an issue or pull request to update docs if this PR changes settings, environment variables, new feature, etc. N/A

@zivkan zivkan requested a review from a team as a code owner October 15, 2025 00:06
@zivkan zivkan requested review from jebriede and nkolev92 October 15, 2025 00:06
Copy link
Copy Markdown
Contributor

@jeffkl jeffkl left a comment

Choose a reason for hiding this comment

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

:shipit:

@Nigusu-Allehu
Copy link
Copy Markdown
Member

backport to release/7.0.x

@Nigusu-Allehu
Copy link
Copy Markdown
Member

backport to release/7.0.x

just testing

@zivkan
Copy link
Copy Markdown
Member Author

zivkan commented Oct 15, 2025

I don't think it will work until after it's merged. The workflow always uses the default branch's yml, not the PR's source branch.

@zivkan zivkan merged commit b9f6f0b into dev Oct 15, 2025
17 of 18 checks passed
@zivkan zivkan deleted the dev-zivkan-fix-backport-bot branch October 15, 2025 01:46
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