Fix deploy workflow #18
Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes the deploy workflow by adding the required name and path parameters to the actions/download-artifact@v5 action. Without these parameters, the v5 version of the action would fail to properly download the artifact created by the build job.
Key Changes
- Added
name: artifactparameter to specify which artifact to download (matching the default name from the upload step) - Added
path: artifactparameter to specify where to extract the artifact contents
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - uses: actions/checkout@v5 | ||
| - uses: actions/download-artifact@v5 | ||
| with: | ||
| name: artifact |
There was a problem hiding this comment.
[nitpick] The artifact name "artifact" matches the default name used by upload-artifact@v5 at line 33 (which doesn't explicitly specify a name). However, for better clarity and maintainability, consider adding an explicit name parameter to the upload step as well. This makes the artifact naming contract between upload and download explicit rather than relying on defaults.
There was a problem hiding this comment.
@copilot open a new pull request to apply changes based on this feedback
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
No description provided.