Skip to content

Split artifact upload and clarify CI/packaging docs#8

Merged
mkrueger merged 2 commits into
dev/mkrueger/workflowsfrom
copilot/sub-pr-4-again
Mar 20, 2026
Merged

Split artifact upload and clarify CI/packaging docs#8
mkrueger merged 2 commits into
dev/mkrueger/workflowsfrom
copilot/sub-pr-4-again

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 20, 2026

Review feedback on the GitHub Actions workflows PR requested better artifact availability on failures and clearer documentation of the CI split.

Changes

  • package-unsigned.yml: Split the single upload step into two:

    • Upload test results — runs if: always() with if-no-files-found: ignore so test logs are always captured even when tests or pack steps fail
    • Upload unsigned artifacts — uploads out/ only on success with if-no-files-found: error (preserves fail-fast for build outputs)
  • README.md: Rewrote the CI section to accurately describe the two-tier setup:

    • GitHub Actions → PR validation + unsigned package creation (on tags)
    • Azure Pipelines (CosmosDB-Shell-Official.yml) → signing and publishing via internal Azure setup

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI mentioned this pull request Mar 20, 2026
@mkrueger mkrueger marked this pull request as ready for review March 20, 2026 12:20
Co-authored-by: mkrueger <341098+mkrueger@users.noreply.github.com>
Copilot AI changed the title [WIP] Add GitHub workflows for building Split artifact upload and clarify CI/packaging docs Mar 20, 2026
Copilot AI requested a review from mkrueger March 20, 2026 12:23
@mkrueger mkrueger merged commit 3ac59d1 into dev/mkrueger/workflows Mar 20, 2026
1 check passed
@mkrueger mkrueger deleted the copilot/sub-pr-4-again branch March 20, 2026 12:23
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