You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Add .clang-tidy to suppress readability-uppercase-literal-suffix in tests/ #181
CodeRabbit repeatedly flags lowercase float-literal suffixes (0.1f, 1.0f, etc.) in test files as triggering readability-uppercase-literal-suffix. The entire codebase consistently uses lowercase f — this is an established project convention.
Proposed Fix
Add a .clang-tidy file that suppresses readability-uppercase-literal-suffix for the tests/ directory, or project-wide if the team prefers.
Why Not Fix Inline
Changing to uppercase F would be inconsistent with every other float literal in the codebase. This is a tooling configuration issue, not a code issue.
Raised by CodeRabbit reviews on PR #179 (rounds 6, 7).
Activity
fix: address CodeRabbit round 7 review findings
fix: CF16 encoder, cloak serialization, energy decloak, tractor drag (#…