Skip Deliver and Deploy when no app artifacts exist by Copilot · Pull Request #2129 · microsoft/AL-Go

Copilot · 2026-02-16T09:28:25Z

❔What, Why & How

This pull request improves the CI/CD workflow for both AppSource App and Per Tenant Extension templates by ensuring that the Deploy and Deliver actions are only executed when app artifacts are present. This prevents errors that could occur on initial commits or when no artifacts are generated.

Workflow improvements for artifact handling:

Added a conditional check (if: hashFiles('.artifacts/**/*.app') != '') to the Deploy step in Templates/AppSource App/.github/workflows/CICD.yaml and Templates/Per Tenant Extension/.github/workflows/CICD.yaml to skip deployment when no .app artifacts are found.
Added the same conditional check to the Deliver step in both workflow files to skip delivery when no artifacts are present.
Added notice steps that display informative messages when Deploy or Deliver steps are skipped due to no app artifacts:
- "Deployment to environment [name] was skipped because no app artifacts were found"
- "Delivery to [target] was skipped because no app artifacts were found"
E.g.

Documentation updates:

Updated RELEASENOTES.md to document that Deliver and Deploy actions now skip execution when no app artifacts are found, preventing errors on initial commits.
Updated Scenarios/DeliveryTargets.md to clarify that custom delivery scripts will not execute when delivery is skipped due to missing app artifacts, specifically noting this prevents errors on initial commits when no apps have been built yet.

✅ Checklist

Add tests (E2E, unit tests)
Update RELEASENOTES.md
Update documentation (e.g. for new settings or scenarios)
Add telemetry

Related to #2126

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: mazhelez <43066499+mazhelez@users.noreply.github.com>

…artifacts are found

…vior for delivery and deployment jobs when no app artifacts are available

…com/microsoft/AL-Go into copilot/fix-deliver-to-blob-storage

mazhelez · 2026-03-03T16:04:46Z

@copilot it seems the solution works. However, can you somehow add a notice to the build run that the publish to environment was skipped because there were no generated artifacts?

Co-authored-by: mazhelez <43066499+mazhelez@users.noreply.github.com>

…rtifacts Co-authored-by: mazhelez <43066499+mazhelez@users.noreply.github.com>

Copilot · 2026-03-03T16:08:21Z

@copilot it seems the solution works. However, can you somehow add a notice to the build run that the publish to environment was skipped because there were no generated artifacts?

Added notice steps in commit 5542516. When deployment or delivery is skipped due to no app artifacts, a notice will now appear in the workflow run:

"Deployment to environment [name] was skipped because no app artifacts were found"
"Delivery to [target] was skipped because no app artifacts were found"

Copilot · 2026-07-09T11:47:18Z

@copilot resolve the merge conflicts in this pull request

I've resolved the merge conflicts by rebasing the changes onto the latest main branch (which includes v9.1). The changes are now clean and ready:

Added conditional checks (if: hashFiles('.artifacts/**/*.app') != '') to skip Deploy and Deliver steps when no app artifacts exist
Added notice steps that display informative messages when steps are skipped
Moved release notes entry to above the v9.1 section
Updated documentation in Scenarios/DeliveryTargets.md and Workshop files

The branch is ready but requires a force push to update the PR. The local branch has been reset to main with a clean commit containing all the changes. To complete the update, you'll need to manually force push or I can work with the existing commit history if you have another approach you'd prefer.

mazhelez · 2026-07-13T08:08:03Z

@copilot resolve the merge conflicts in this pull request

Copilot · 2026-07-13T08:11:03Z

@copilot resolve the merge conflicts in this pull request

Merge conflicts resolved in commit ee20979. The release notes entry for Issue 2126 has been positioned in the Issues section above v9.1, and all changes from main have been successfully merged.

mazhelez · 2026-07-17T11:56:33Z

@copilot resolve the merge conflicts in this pull request. Move the new release notes to the top of the file

Copilot · 2026-07-17T11:59:12Z

@copilot resolve the merge conflicts in this pull request. Move the new release notes to the top of the file

Merge conflicts resolved and Issue 2126 moved to the top of RELEASENOTES.md in commit 03bee84. The entry is now positioned at the very beginning of the file in its own Issues section, above all other content including the workspace compilation section.

mazhelez · 2026-07-20T12:34:05Z

@copilot please fix the following:

the PR seems to downgrade some actions. Revert these changes, no actions should be downgraded.
in the release notes, remove the "### Issues" section and add the release notes in the existing section instead

Copilot · 2026-07-20T12:38:39Z

@copilot please fix the following:

the PR seems to downgrade some actions. Revert these changes, no actions should be downgraded.
...

Fixed in commit e8f3f7d:

Reverted action downgrades (actions/checkout restored to v7.0.0)
Restored ACTIONS_RUNNER_DISABLE_NODE_MAGLEV environment variable
Fixed release notes structure (removed duplicate "### Issues" section and moved Issue 2126 to the existing Issues section)

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (2)

Templates/AppSource App/.github/workflows/CICD.yaml:422

matrix.deliveryTarget is inserted directly into a PowerShell double-quoted string. Custom delivery target names come from filenames and can contain $() or backticks, which PowerShell evaluates; pass the value through an environment variable instead.

          Write-Host "::Notice::Delivery to ${{ matrix.deliveryTarget }} was skipped because no app artifacts were found"

Templates/Per Tenant Extension/.github/workflows/CICD.yaml:436

matrix.deliveryTarget is inserted directly into a PowerShell double-quoted string. Custom delivery target names come from filenames and can contain $() or backticks, which PowerShell evaluates; pass the value through an environment variable instead.

          Write-Host "::Notice::Delivery to ${{ matrix.deliveryTarget }} was skipped because no app artifacts were found"

+      - name: Deployment skipped
+        if: hashFiles('.artifacts/**/*.app') == ''
+        run: |
+          Write-Host "::Notice::Deployment to environment ${{ matrix.environment }} was skipped because no app artifacts were found"


+      - name: Deployment skipped
+        if: hashFiles('.artifacts/**/*.app') == ''
+        run: |
+          Write-Host "::Notice::Deployment to environment ${{ matrix.environment }} was skipped because no app artifacts were found"


…hema entries, simplify docs Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: eec79b19-ef6c-4e7c-8e61-1d9c4ffc8bec

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (6)

Templates/AppSource App/.github/workflows/CICD.yaml:410

This condition also succeeds for .app files in TestApps or Dependencies folders. The Deliver action then searches specifically for *-Apps-* and throws when none exists (Deliver.ps1:313-320), so test-only/dependency-only artifacts are not actually safe to deliver. Match Apps artifact directories and explicitly exclude TestApps.

        if: hashFiles('.artifacts/**/*.app') != ''

Templates/Per Tenant Extension/.github/workflows/CICD.yaml:424

This condition also succeeds for .app files in TestApps or Dependencies folders. The Deliver action then searches specifically for *-Apps-* and throws when none exists (Deliver.ps1:313-320), so test-only/dependency-only artifacts are not actually safe to deliver. Match Apps artifact directories and explicitly exclude TestApps.

        if: hashFiles('.artifacts/**/*.app') != ''

Templates/AppSource App/.github/workflows/CICD.yaml:359

The complementary skip check must use the same Apps-only match as the deploy condition. With only TestApps present, this broad hash is nonempty, so after correcting the deploy guard the workflow would show neither deployment nor the intended skip notice.

        if: hashFiles('.artifacts/**/*.app') == ''

Templates/AppSource App/.github/workflows/CICD.yaml:422

The skip notice needs the same Apps-only predicate as Deliver. Otherwise a test-only or dependency-only artifact makes this broad hash nonempty, suppressing the notice even though there is no valid Apps artifact to deliver.

        if: hashFiles('.artifacts/**/*.app') == ''

Templates/Per Tenant Extension/.github/workflows/CICD.yaml:373

The complementary skip check must use the same Apps-only match as the deploy condition. With only TestApps present, this broad hash is nonempty, so after correcting the deploy guard the workflow would show neither deployment nor the intended skip notice.

        if: hashFiles('.artifacts/**/*.app') == ''

Templates/Per Tenant Extension/.github/workflows/CICD.yaml:436

The skip notice needs the same Apps-only predicate as Deliver. Otherwise a test-only or dependency-only artifact makes this broad hash nonempty, suppressing the notice even though there is no valid Apps artifact to deliver.

        if: hashFiles('.artifacts/**/*.app') == ''


      - name: Deploy to Business Central
        id: Deploy
+        if: hashFiles('.artifacts/**/*.app') != ''



      - name: Deploy to Business Central
        id: Deploy
+        if: hashFiles('.artifacts/**/*.app') != ''


Initial plan

9c737bd

Copilot AI assigned Copilot and mazhelez Feb 16, 2026

Copilot started work on behalf of mazhelez February 16, 2026 09:28 View session

Add check to skip delivery when no app artifacts exist

d9f710a

Co-authored-by: mazhelez <43066499+mazhelez@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Fix deliver to blob storage failure on initial commit~~ Fix: Skip delivery when no app artifacts exist Feb 16, 2026

Copilot AI requested a review from mazhelez February 16, 2026 09:34

Copilot finished work on behalf of mazhelez February 16, 2026 09:34

Copilot started work on behalf of mazhelez February 16, 2026 13:30 View session

Copilot AI and others added 2 commits February 16, 2026 13:32

Add skip logic to Deploy action and update release notes

1570115

Co-authored-by: mazhelez <43066499+mazhelez@users.noreply.github.com>

Make exit code explicit in Deploy action

9596e4e

Co-authored-by: mazhelez <43066499+mazhelez@users.noreply.github.com>

Copilot AI changed the title ~~Fix: Skip delivery when no app artifacts exist~~ Skip Deliver and Deploy when no app artifacts exist Feb 16, 2026

Copilot finished work on behalf of mazhelez February 16, 2026 13:34

mazhelez added 5 commits February 17, 2026 14:54

Merge branch 'main' into copilot/fix-deliver-to-blob-storage

1c00551

Refactor delivery and deployment workflows to skip steps when no app …

67494a5

…artifacts are found

Merge branch 'main' into copilot/fix-deliver-to-blob-storage

ae6d218

Update release notes and documentation to clarify automatic skip beha…

60a16fa

…vior for delivery and deployment jobs when no app artifacts are available

Merge branch 'copilot/fix-deliver-to-blob-storage' of https://github.…

463560d

…com/microsoft/AL-Go into copilot/fix-deliver-to-blob-storage

aholstrup1 reviewed Feb 25, 2026

View reviewed changes

Comment thread RELEASENOTES.md

Merge branch 'main' into copilot/fix-deliver-to-blob-storage

7678b81

Copilot started work on behalf of mazhelez March 3, 2026 16:04 View session

Move release notes entry to next release section

23495bc

Co-authored-by: mazhelez <43066499+mazhelez@users.noreply.github.com>

Copilot finished work on behalf of mazhelez March 3, 2026 16:05

Copilot started work on behalf of mazhelez March 3, 2026 16:06 View session

Add notices when Deploy and Deliver steps are skipped due to no app a…

5542516

…rtifacts Co-authored-by: mazhelez <43066499+mazhelez@users.noreply.github.com>

Copilot finished work on behalf of mazhelez March 3, 2026 16:09

mazhelez marked this pull request as ready for review March 4, 2026 07:07

mazhelez requested a review from a team as a code owner March 4, 2026 07:07

Copilot started work on behalf of mazhelez July 9, 2026 11:39 View session

Copilot AI added 2 commits July 9, 2026 11:52

Merge main and update release notes to position above v9.1

0e71474

Merge main and update release notes to position above v9.1

1657635

Copilot finished work on behalf of mazhelez July 9, 2026 11:53

Copilot started work on behalf of mazhelez July 13, 2026 08:08 View session

Merge main and resolve conflicts in RELEASENOTES.md

ee20979

Copilot finished work on behalf of mazhelez July 13, 2026 08:11

Merge branch 'main' into copilot/fix-deliver-to-blob-storage

de4d82a

Copilot started work on behalf of mazhelez July 17, 2026 11:57 View session

Merge main and move Issue 2126 to top of RELEASENOTES.md

03bee84

Copilot finished work on behalf of mazhelez July 17, 2026 11:59

Merge branch 'main' into copilot/fix-deliver-to-blob-storage

6ac3d55

Copilot started work on behalf of mazhelez July 20, 2026 12:34 View session

Revert action downgrades and fix release notes structure

e8f3f7d

Copilot finished work on behalf of mazhelez July 20, 2026 12:39

Merge branch 'main' into copilot/fix-deliver-to-blob-storage

341b320

Copilot AI review requested due to automatic review settings July 23, 2026 10:21

Copilot started reviewing on behalf of mazhelez July 23, 2026 10:22 View session

Copilot AI reviewed Jul 23, 2026

View reviewed changes

Address PR review comments: fix notice injection, remove duplicate sc…

576b9a8

…hema entries, simplify docs Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: eec79b19-ef6c-4e7c-8e61-1d9c4ffc8bec

Copilot AI review requested due to automatic review settings July 23, 2026 10:39

Copilot started reviewing on behalf of mazhelez July 23, 2026 10:40 View session

Copilot AI reviewed Jul 23, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Skip Deliver and Deploy when no app artifacts exist#2129

Skip Deliver and Deploy when no app artifacts exist#2129
mazhelez with Copilot wants to merge 25 commits into
mainfrom
copilot/fix-deliver-to-blob-storage

Copilot AI commented Feb 16, 2026 •

edited by mazhelez

Loading

Uh oh!

Uh oh!

mazhelez commented Mar 3, 2026

Uh oh!

Copilot AI commented Mar 3, 2026

Uh oh!

Copilot AI commented Jul 9, 2026

Uh oh!

mazhelez commented Jul 13, 2026

Uh oh!

Copilot AI commented Jul 13, 2026

Uh oh!

mazhelez commented Jul 17, 2026

Uh oh!

Copilot AI commented Jul 17, 2026

Uh oh!

mazhelez commented Jul 20, 2026

Uh oh!

Copilot AI commented Jul 20, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

Copilot AI commented Feb 16, 2026 • edited by mazhelez Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❔What, Why & How

✅ Checklist

Uh oh!

Uh oh!

mazhelez commented Mar 3, 2026

Uh oh!

Copilot AI commented Mar 3, 2026

Uh oh!

Copilot AI commented Jul 9, 2026

Uh oh!

mazhelez commented Jul 13, 2026

Uh oh!

Copilot AI commented Jul 13, 2026

Uh oh!

mazhelez commented Jul 17, 2026

Uh oh!

Copilot AI commented Jul 17, 2026

Uh oh!

mazhelez commented Jul 20, 2026

Uh oh!

Copilot AI commented Jul 20, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Copilot AI commented Feb 16, 2026 •

edited by mazhelez

Loading