Add error handling for missing packages in publish workflow #186
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: JDetmar <[email protected]>
Copilot
AI
changed the title
[WIP] Update CI steps and publish workflow for NLog.Extensions.AzureStorage
Add error handling for missing packages in publish workflow
Dec 13, 2025
JDetmar
approved these changes
Dec 13, 2025
Contributor
There was a problem hiding this comment.
Pull request overview
This PR enhances error handling in the publish workflow to prevent silent failures when package artifacts are missing or download failures occur.
Key Changes:
- Added validation to verify the
packages/directory exists before scanning - Added validation to ensure at least one
.nupkgfile is present - Implemented fail-fast behavior with descriptive error messages written to stderr
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
JDetmar
added a commit
that referenced
this pull request
Dec 13, 2025
#185) * Add CI steps for restoring projects and create publish workflow for NuGet * Update .github/workflows/publish.yml Co-authored-by: Copilot <[email protected]> * Update .github/workflows/publish.yml Co-authored-by: Copilot <[email protected]> * Update .github/workflows/publish.yml Co-authored-by: Copilot <[email protected]> * Update .github/workflows/ci.yml Co-authored-by: Copilot <[email protected]> * Update .github/workflows/ci.yml Co-authored-by: Copilot <[email protected]> * Add error handling for missing packages in publish workflow (#186) * Initial plan * Add error handling for package discovery in publish workflow Co-authored-by: JDetmar <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: JDetmar <[email protected]> * Fix invalid matrix strategy placement in publish workflow (#187) * Initial plan * Fix publish.yml syntax error - move strategy matrix to proper structure Co-authored-by: JDetmar <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: JDetmar <[email protected]> * Add restore step for matrix projects in CI workflow --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Copilot <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The
findcommand in the publish workflow silently succeeds when no packages exist, masking artifact download failures or incorrect run IDs.Changes
Added validation to both package listing steps:
packages/directory exists before scanning.nupkgfile is presentApplied identical validation to the dry run summary step.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.