Update ruling results for PR #6555#6733
Conversation
🤖 Generated with GitHub Actions
SummaryAhh, here we have a tally of the seas adjusted, mate. Rule S7778 — the one concernin' methods that be combinable, savvy? — has been set straight in PR What reviewers should knowThe changes be purely in the expected test results — no code changes, just the test expectations updated to align with PR #6555's fix. Start by understandin' that rule S7778 flags method chains, and the issue was overzealous flaggin' of methods that only accept a single argument (which cannot be combined in the manner the rule was warnin' about). Each JSON file represents expected violations per project: additions mean the rule now correctly catches violations it missed before, removals mean false positives have been eliminated. The Ghost project shows the most substantial adjustments — likely a codebase with many test cases. If ye need to verify the actual rule change, that be in PR #6555; this PR is merely ensurin' the test suite knows what violations to expect from the updated rule.
|
There was a problem hiding this comment.
LGTM! ✅
A clean and correct auto-generated update, mate — nothing aboard this vessel that ought to give a reviewer pause. The ruling snapshots reflect precisely the adjusted reach of the S7778 rule as fixed in PR #6555, and the ledger balances true.
|
No longer needed — the original PR is now up to date. |
Auto-generated ruling update for PR #6555.
🤖 Generated with GitHub Actions