Skip to content

Comments

fix: Fix download of build artifacts on relese workflow#179

Merged
Blackhex merged 1 commit intoa2aproject:mainfrom
Blackhex:fix/release
Sep 4, 2025
Merged

fix: Fix download of build artifacts on relese workflow#179
Blackhex merged 1 commit intoa2aproject:mainfrom
Blackhex:fix/release

Conversation

@Blackhex
Copy link
Collaborator

@Blackhex Blackhex commented Sep 4, 2025

After update, the actions/download-artifact do not place the downloaded artifacts to build-artifacts folder as before. This PR sets the location of the artifact explicitly. Furthermore, it requests only the build-artifacts artifact which removes warning.

Fix validated by https://github.com/Blackhex/a2a-dotnet/actions/runs/17468209270

@Blackhex Blackhex requested a review from Copilot September 4, 2025 15:03
@Blackhex Blackhex marked this pull request as ready for review September 4, 2025 15:09
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

This PR fixes the download location of build artifacts in the GitHub release workflow by explicitly configuring the actions/download-artifact action. The update addresses a breaking change where the action no longer automatically places artifacts in the expected build-artifacts folder.

  • Adds explicit name and path configuration to three download-artifact steps
  • Sets the download path to ${{ github.workspace }}/build-artifacts for consistency
  • Specifies build-artifacts as the target artifact name to eliminate warnings
Comments suppressed due to low confidence (1)

.github/workflows/release.yaml:1

  • There's a typo in the title: 'relese' should be 'release'.
# Publish new package versions of A2A .NET SDK

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Blackhex Blackhex added this pull request to the merge queue Sep 4, 2025
Merged via the queue into a2aproject:main with commit 275a7e0 Sep 4, 2025
6 checks passed
@Blackhex Blackhex deleted the fix/release branch September 4, 2025 15:35
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