Skip to content

Remove Azure-dependent CI steps to fix forked PR builds#1154

Merged
jbogard merged 2 commits intomainfrom
copilot/fix-ci-failures-azure-steps
Feb 22, 2026
Merged

Remove Azure-dependent CI steps to fix forked PR builds#1154
jbogard merged 2 commits intomainfrom
copilot/fix-ci-failures-azure-steps

Conversation

Copy link
Contributor

Copilot AI commented Feb 22, 2026

Forked PRs fail CI because Azure secrets (AZURE_CLIENT_ID, AZURE_TENANT_ID, etc.) are not available to fork workflows, causing azure/login@v2 to abort the entire job.

Changes

  • Removed Azure Login via OIDC step (azure/login@v2)
  • Removed Sign packages step (NuGetKeyVaultSignTool with Azure Key Vault)

All other steps — checkout, dotnet setup, build/test, push, and artifact upload — remain intact.

Original prompt

Create a pull request in LuckyPennySoftware/MediatR to fix CI failures on forked pull requests by removing Azure-dependent steps entirely (no conditional execution).

Context:

Required changes:

  1. Edit .github/workflows/ci.yml to remove the following steps from the build job:
    • Azure Login via OIDC step using azure/login@v2 (client-id / tenant-id / subscription-id inputs)
    • Sign packages step that runs NuGetKeyVaultSignTool sign ... --azure-key-vault-managed-identity ...
  2. Do not add if: conditions or other gating logic; the steps should be deleted.
  3. Ensure the workflow still builds/tests and uploads artifacts successfully.

Notes:

  • Leave other steps intact (checkout, setup-dotnet, install tool, build/test, push, upload artifact).
  • Verify YAML remains valid and indentation correct after removing steps.

This pull request was created from Copilot chat.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: jbogard <104498+jbogard@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI failures on forked pull requests by removing Azure steps Remove Azure-dependent CI steps to fix forked PR builds Feb 22, 2026
Copilot AI requested a review from jbogard February 22, 2026 15:58
@jbogard jbogard marked this pull request as ready for review February 22, 2026 15:59
@jbogard jbogard merged commit d67f2d5 into main Feb 22, 2026
5 checks passed
@jbogard jbogard deleted the copilot/fix-ci-failures-azure-steps branch February 22, 2026 16:01
This was referenced Mar 8, 2026
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