Skip to content

Fix release publish failure by normalizing nested artifact paths - #4

Closed
TaylanTatli with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-publish-github-release-job
Closed

Fix release publish failure by normalizing nested artifact paths#4
TaylanTatli with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-publish-github-release-job

Conversation

Copilot AI commented Jul 28, 2026

Copy link
Copy Markdown

Publish GitHub release failed because release assets downloaded by actions/download-artifact were stored in nested directories, while the normalization step only searched the artifact root. As a result, required assets (e.g., .deb) were not discovered and release creation aborted.

  • Root cause addressed: recursive asset discovery

    • Updated release asset normalization to search the full artifact tree instead of only top-level files.
    • This aligns with the actual structure produced by merged artifact downloads (deb/, rpm/, appimage/, binary/, etc.).
  • Release contract preserved

    • Existing strict checks remain: exactly one asset is required per expected pattern.
    • Output naming remains unchanged (SCADmate_<version>_<os>_<arch>...) so downstream release expectations stay stable.
  • Relevant change

    # before
    find "$asset_dir" -maxdepth 1 -type f -name "$pattern" -print0
    
    # after
    find "$asset_dir" -type f -name "$pattern" -print0

Copilot AI changed the title [WIP] Fix failing GitHub Actions job 'Publish GitHub release' Fix release publish failure by normalizing nested artifact paths Jul 28, 2026
Copilot AI requested a review from TaylanTatli July 28, 2026 05:03
@TaylanTatli
TaylanTatli marked this pull request as ready for review July 28, 2026 05:04
@TaylanTatli
TaylanTatli deleted the copilot/fix-publish-github-release-job branch July 28, 2026 05:04
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