Skip to content

Conversation

@soniseth0
Copy link
Contributor

remove explicit == true/false comparisons in filters_test.go, keep the test expectations intact while aligning with idiomatic Go style, eliminate redundant boolean logic that could trip linters or reviewers

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors boolean assertions in filters_test.go to follow idiomatic Go style by removing redundant explicit comparisons with true and false. The changes replace verbose comparisons like if condition == false with the more concise if !condition and if condition == true with if condition, improving code readability while maintaining identical test behavior.

Key changes:

  • Converted all == false comparisons to use the ! negation operator
  • Converted all == true comparisons to direct boolean evaluation
  • No changes to test logic or expectations

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@soniseth0 soniseth0 closed this by deleting the head repository Nov 21, 2025
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