Skip to content

Update BCArtifact to 29.0.52664.0 & unblock platform uptake with AL1430 ruleset exception#9593

Merged
mazhelez merged 3 commits into
mainfrom
automation/main/updatebcartifact/2607170642
Jul 20, 2026
Merged

Update BCArtifact to 29.0.52664.0 & unblock platform uptake with AL1430 ruleset exception#9593
mazhelez merged 3 commits into
mainfrom
automation/main/updatebcartifact/2607170642

Conversation

@krupybalu

@krupybalu krupybalu commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following changes:

Update BCArtifact version. New value: 29.0.52664.0
Update platform version. New value: 29.0.52663.0

Ruleset exception for AL1430.

AB#642468
AB#539394

@krupybalu
krupybalu requested review from a team July 20, 2026 08:52
@github-actions github-actions Bot added Build: Automation Workflows and other setup in .github folder Build: scripts & configs Build scripts and configuration files Other GitHub request for other area than SCM, Finance or Integration Ownership: Needs Review Ownership is Other, low confidence, or needs manual correction labels Jul 20, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 20, 2026
@github-actions

Copy link
Copy Markdown
Contributor

$\textbf{🟡\ Medium\ Severity\ —\ Agent}$

src/rulesets/ruleset.json is no longer valid JSON: the newly added AL1430 rule entry is followed by a trailing comma before the closing ']' of the rules array. This file is $include'd directly by src/rulesets/base.ruleset.json and consumed as-is by the AL compiler's ruleset loader (no JSON5/JSONC leniency), so any build or compile that loads this ruleset (e.g. via PreCompileApp.ps1) will fail to parse it, breaking analyzer configuration for every project that references base.ruleset.json.

Recommendation:

  • remove the trailing comma after the AL1430 object.

Suggested fix (apply manually — could not be anchored as a one-click suggestion):

    {
        "id": "AL1430",
        "action": "Warning",
        "justification": "TODO(#642602) - Recent bugfix uncovered violations that need to be addressed."
    }

Line mapping was unavailable, so this was posted as an issue comment.

👍 useful · ❤️ especially valuable · 👎 wrong - reply with why

@onbuyuka
onbuyuka enabled auto-merge July 20, 2026 13:29
@mazhelez
mazhelez disabled auto-merge July 20, 2026 15:19
@mazhelez
mazhelez merged commit 0abbd4e into main Jul 20, 2026
662 of 674 checks passed
@mazhelez
mazhelez deleted the automation/main/updatebcartifact/2607170642 branch July 20, 2026 15:19
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 Build: scripts & configs Build scripts and configuration files Other GitHub request for other area than SCM, Finance or Integration Ownership: Needs Review Ownership is Other, low confidence, or needs manual correction

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants