Fix #484: skipping behavior for multi-location errors

[kluhan]

Changes

• Fixed @rpglint-skip to properly suppress errors with multiple locations.
• Introduced pushErrorIfNotSkipped to simplify error handling across existing rules.

Tests

• Added unit tests for skip behavior with repeated literals.
• Verified skipped ranges don’t suppress unrelated errors.

Why It Matters

Previously, some errors (like variable casing) weren’t suppressed on second-pass checks. This unifies skip logic, ensures consistent suppression, and enables future directives like @rpglint-skip: StringLiteralDupe without extra complexity.

[kluhan]

@worksofliam Could you help clarify the difference between @rpglint-skip-rules and @rpglint-skip?
I’ve checked the documentation but couldn’t find anything, and from what I can see in the code, they appear to behave the same.
Currently, my fix applies only to @rpglint-skip.

[worksofliam]

@kluhan

To me it's like this:

• @rpglint-skip skips both rules and indentation checks
• @rpglint-skip-rules skips all rules but indentation
• @rpglint-skip-indent skils indent checking, but checks all other rules