Repository navigation
[main@1a1f9b0] Update AL-Go System Files from microsoft/AL-Go-PTE@preview - 506dd14 / Related to AB#640900 - #13132
Closed
business-central-bot[bot] wants to merge 3 commits into
Closed
business-central-bot[bot] wants to merge 3 commits into
business-central-bot[bot] wants to merge 3 commits into
Conversation
…view - 506dd14 / Related to AB#640900
Darrick (darjoo)
previously approved these changes
Oct 22, 2026
Maria Zhelezova (mazhelez)
previously approved these changes
Oct 22, 2026
Alexander Holstrup (aholstrup1)
approved these changes
Oct 22, 2026
Alexander Holstrup (aholstrup1)
requested changes
Oct 22, 2026
| @@ -1,5 +1,5 @@ | |||
| { | |||
| "$schema": "https://raw.githubusercontent.com/microsoft/AL-Go/20620d1a3676cecfa2d0bbba3ced4a89f0e08217/Actions/.Modules/settings.schema.json", | |||
| "$schema": "https://raw.githubusercontent.com/microsoft/AL-Go/959519dfe53da49b83bd5cf1f43eaa0be7ca5301/Actions/.Modules/settings.schema.json", | |||
Contributor
There was a problem hiding this comment.
We need to wait with this PR until we have uptaken the new settings so we don't accidently disable the code reviewer
Related to AB#640900 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 584d2144-5547-4d2f-b687-de5c520c3892
Wael (WaelAbuSeada)
dismissed stale reviews from Maria Zhelezova (mazhelez) and Darrick (darjoo)
via
October 22, 2026 18:51
c626afc
Wael (WaelAbuSeada)
had a problem deploying
to
Official-Build
October 22, 2026 18:53 — with
GitHub Actions
Failure
Use the published AL-Go trigger generator at 959519dfe53da49b83bd5cf1f43eaa0be7ca5301. Related to AB#640900. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 584d2144-5547-4d2f-b687-de5c520c3892
Alexander Holstrup (aholstrup1)
approved these changes
Oct 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
preview
Note that when using the preview version of AL-Go for GitHub, we recommend you Update your AL-Go system files, as soon as possible when informed that an update is available.
Copilot code review for AL pull requests
AL-Go repositories can now opt in to AI-powered review of AL pull requests, with configurable models, target branches, excluded authors, and BCQuality skills and knowledge. See Enable Copilot code review for setup, organization policy requirements, and operational details.
Issues
Issue 2398 - Preserve dependency paths containing commas when passing a single app to Run-AlPipeline. Download Project Dependencies now warns with the complete path if a resolved dependency file is missing, without changing the output lists.
Issue 2390 - AnalyzeTests no longer crashes under strict mode when a page scripting test suite contains both passed and failed tests, preserving the failure summary.
Issue 2387 - Deployment environment discovery now paginates GitHub API results, so Publish To Environment and CI/CD can discover environments beyond the first 30 and read their branch policies.
Related to AB#640900
BCApps reviewer policy carried forward
This uptake migrates BCApps from the reusable
microsoft/BC-ALAgentsreviewer to native AL-Go. The policy must be explicit in.github/AL-Go-Settings.json; copying the raw preview workflows alone leaves automatic reviews disabled.CodeReviewAgent.enabled: true; target branchmain, matching the existing intake rather than widening review scope to all PR-build branches.business-central-bot[bot], preserving the existing automation-author exclusion. Native review context also skips draft PRs.gpt-5.6-soland leaf modelgpt-5.6-luna, matching the previous reusable workflow defaults.https://github.com/microsoft/BCQuality, content version1.7supplied by this AL-Go preview (a deliberate update from the old reusable engine's pinned 1.6 content).microsoftlayer andmicrosoft/knowledge/**; no disabled skills or denied articles, preserving the existing Microsoft-only content policy rather than inheriting AL-Go's all-layer default.Generated workflow activation
The official updater attempt 39835004965 was rejected before the update job ran because the PR branch is not allowed to deploy to
Official-Build. No environment protection, approval, secret, or repository policy was changed.The two enabled review triggers were instead generated locally using the exact published AL-Go YAML class and trigger-generation code at
959519dfe53da49b83bd5cf1f43eaa0be7ca5301, then committed normally to this PR. This is a secret-free code-generation operation, not a deployment. The generated workflows are the published templates plus only the three-linepull_requestand three-lineworkflow_runtrigger additions; future AL-Go updates retain them through the checked-in settings.The native adapter retains the previous reviewer behavior defaults: Medium minimum severity, 25 findings per domain, serial leaves, 30-minute CLI timeout, parse errors surfaced, and no overview summary comment. These are engine defaults, not unsupported repository settings.
Additional scope on top of the original updater PR: three files, 31 added lines (25 settings lines and six generated trigger lines). Settings were validated against the exact pinned schema, all other BCApps settings were compared unchanged, and both generated workflows were checked against the published templates plus their enabled triggers. The preview template revision is unchanged. This PR has not been merged.