Skip to content

[28.x] Consolidate VerifyAppChanges into PullRequestHandler custom job - #12616

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

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

Conversation

@mazhelez

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

Copy link
Copy Markdown
Contributor

This aligns releases/28.x with main/releases/29.x by moving the standalone VerifyAppChanges workflow into PullRequestHandler.yaml as the CustomJob-VerifyAppChanges AL-Go custom job.

Changes

  • PullRequestHandler.yaml:
    • Build1 job now gates on the custom job: needs: [ Initialization, CustomJob-VerifyAppChanges ]
    • Build job now gates on the custom job: needs: [ Initialization, Build1, CustomJob-VerifyAppChanges ]
    • Added the CustomJob-VerifyAppChanges job (after StatusCheck) that runs this branch's existing two validation actions: TestPreprocessorSymbols and TestObjectIdsAndManifests.
  • Deleted the standalone .github/workflows/VerifyAppChanges.yaml.

The branch's existing validation action set is preserved exactly (only TestPreprocessorSymbols and TestObjectIdsAndManifests — no VerifyMiappSync or ValidateCountryProjects, which are not validated on this branch).

Reference: AB#640789

Align releases/28.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.
Preserves this branch's two validation actions and removes 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

This branch was successfully deployed

1 active deployment
triage — cc61a427 Deployed Oct 9, 2026 by mazhelez via Classify team ownership #7343
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