Skip to content

Clean up projects.json and groups.json#9013

Open
aholstrup1 wants to merge 2 commits into
mainfrom
private/aholstrup/cleanup_projectsjson
Open

Clean up projects.json and groups.json#9013
aholstrup1 wants to merge 2 commits into
mainfrom
private/aholstrup/cleanup_projectsjson

Conversation

@aholstrup1

@aholstrup1 aholstrup1 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

What & why

Clean up projects.json and groups.json so each project only appears in one file

Linked work

Fixes #

How I validated this

  • I read the full diff and it contains only changes I intended.
  • I built the affected app(s) locally with no new analyzer warnings.
  • I ran the change in Business Central and confirmed it behaves as expected.
  • I added or updated tests for the new behavior, or explained below why none are needed.

What I tested and the outcome (required — be specific: scenarios, commands, screenshots for UI changes)

Risk & compatibility

@aholstrup1 aholstrup1 requested review from a team July 2, 2026 06:28
@github-actions github-actions Bot added the Build: scripts & configs Build scripts and configuration files label Jul 2, 2026
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Could not find a linked ADO work item. Please link one by using the pattern 'AB#' followed by the relevant work item number. You may use the 'Fixes' keyword to automatically resolve the work item when the pull request is merged. E.g. 'Fixes AB#1234'

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Copilot PR Review

Iteration 1 · Outcome: no-knowledge

The PR touches only build/groups.json and build/projects.json (JSON build-pipeline configuration); no .al, page, or control-add-in files are present, so every AL-domain leaf sub-skill found no applicable knowledge or scope to evaluate. The rolled-up finding below comes from the super-skill's own cross-cutting self-review pass, not from a leaf.

Knowledge source: https://github.com/microsoft/BCQuality@822cae1b2771ac25f665f73369f69093bd4fd630

No findings were posted for this iteration.

Orchestrator pre-filter (13 file(s) excluded)

  • layer-disabled (knowledge) : 13 file(s)

Findings produced by the Copilot CLI agent against BCQuality at 822cae1b2771ac25f665f73369f69093bd4fd630. Reply 👎 on any inline comment to flag false positives.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build: scripts & configs Build scripts and configuration files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant