Skip to content

Make AA0189 an error in the base ruleset - #10087

Closed
Magnus Hartvig Grønbech (Groenbech96) wants to merge 2 commits into
mainfrom
copilot/aa0189-error
Closed

Make AA0189 an error in the base ruleset#10087
Magnus Hartvig Grønbech (Groenbech96) wants to merge 2 commits into
mainfrom
copilot/aa0189-error

Conversation

@Groenbech96

@Groenbech96 Magnus Hartvig Grønbech (Groenbech96) commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Removes the AA0189 override from src/rulesets/base.ruleset.json.

The ruleset declares "generalAction": "Error", so removing the explicit "action": "Warning" entry promotes AA0189 to an error for the base application.

@github-actions github-actions Bot added Other GitHub request for other area than SCM, Finance or Integration Ownership: Needs Review Ownership is Other, low confidence, or needs manual correction labels Aug 10, 2026
@github-actions

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'

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: b6f3c139-b1a1-4b39-8864-cffb67ef6212
@JesperSchulz

Copy link
Copy Markdown
Contributor

Nice catch. I made that catch too 😉 As well as 18 others in #10088. Closing this one as a dupe.

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

Labels

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.

2 participants