Skip to content

[26.x] Consolidate VerifyAppChanges into PullRequestHandler custom job - #12618

Closed
Maria Zhelezova (mazhelez) wants to merge 1 commit into
releases/26.xfrom
mazhelez/cleanup-verifyappchanges-26x
Closed

Maria Zhelezova (mazhelez) wants to merge 1 commit into
releases/26.xfrom
mazhelez/cleanup-verifyappchanges-26x

Conversation

@mazhelez

@mazhelez Maria Zhelezova (mazhelez) commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Summary

After the move of all app code to BCApps, VerifyAppChanges should no longer be a standalone workflow — on main and releases/29.x it lives as an AL-Go custom job on the PR Build workflow. This PR applies that same consolidation to releases/26.x.

Changes

  • .github/workflows/PullRequestHandler.yaml
    • Build1 job now needs: [ Initialization, CustomJob-VerifyAppChanges ]
    • Build job now needs: [ Initialization, Build1, CustomJob-VerifyAppChanges ]
    • Added CustomJob-VerifyAppChanges AL-Go custom job (sibling of Build/StatusCheck) that checks out and runs the app-change validation.
  • Deleted the standalone .github/workflows/VerifyAppChanges.yaml.

This branch's standalone workflow validated a single action, so the custom job preserves exactly that one — microsoft/BCApps/.github/actions/TestPreprocessorSymbols — matching this branch's existing validation (no extra actions from other branches were added).

AB#640789

Aligns releases/26.x with main/releases/29.x by moving the standalone VerifyAppChanges workflow into PullRequestHandler.yaml as the CustomJob-VerifyAppChanges AL-Go custom job gating Build1 and Build, preserving this branch's existing TestPreprocessorSymbols validation action, and removing the standalone workflow.

AB#640789

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot added Build: Automation Workflows and other setup in .github folder Ownership: Needs Review Ownership is Other, low confidence, or needs manual correction Team: Other GitHub request for other area than SCM, Finance or Integration labels Oct 9, 2026
@mazhelez
Maria Zhelezova (mazhelez) marked this pull request as ready for review October 9, 2026 12:17
@mazhelez
Maria Zhelezova (mazhelez) requested a review from a team October 9, 2026 12:17
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.


🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

This branch was successfully deployed

1 active deployment
triage — 416ac972 Deployed Oct 9, 2026 by mazhelez via Classify team ownership #7342
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build: Automation Workflows and other setup in .github folder Ownership: Needs Review Ownership is Other, low confidence, or needs manual correction Team: Other GitHub request for other area than SCM, Finance or Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant