Skip to content

Update ruling results for PR #6626#6627

Open
github-actions[bot] wants to merge 4 commits intocodex/ts6-rc-evalfrom
fix/update-ruling-for-codex/ts6-rc-eval
Open

Update ruling results for PR #6626#6627
github-actions[bot] wants to merge 4 commits intocodex/ts6-rc-evalfrom
fix/update-ruling-for-codex/ts6-rc-eval

Conversation

@github-actions
Copy link
Contributor

Auto-generated ruling update for PR #6626.

🤖 Generated with GitHub Actions

@sonar-review-alpha
Copy link

sonar-review-alpha bot commented Mar 18, 2026

Summary

Updates expected ruling test results across multiple analysis rules and projects. Based on changes in PR #6626, violations have been added, removed, or shifted in the baseline results. Changes span numerous rules (S1874, S3403, S3782, S4325, S6551, S6582, S6606, S7728, etc.) and test projects (TypeScript, Ghost, Joust, ace, angular.js, and others). Includes creation of new expected result files and deletion of obsolete ones.

What reviewers should know

All changes are in its/ruling/src/test/expected/ — these are the baseline expected results that ruling integration tests validate against. The modifications represent line-number shifts and additions/removals of detected violations. Verify that the nature and scale of changes align with the rule logic updates in PR #6626. Pay particular attention to files being deleted entirely (e.g., ace/typescript-S1874.json, angular.js/javascript-S2301.json) to ensure those rules are intentionally no longer triggering on those projects.


  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

@vdiez vdiez force-pushed the codex/ts6-rc-eval branch from eecf520 to 75a6a1c Compare March 18, 2026 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant