Skip to content

Conversation

@connor-powers
Copy link
Owner

Context

I didn't like that gcovr was throwing that warning about that line in the utils file, so I wanted to see if simply adjusting the formatting of the line would resolve the issue, and apparently it did.

Changes

  • Adjusted the formatting of the line in question in the utils file
  • Removed the flag on gcovr that I had to add to deal with the parse error
  • Reran test coverage script

Testing

Needs to pass all existing test suites

…pt no longer hit that "suspicious hit" warning
@connor-powers connor-powers self-assigned this May 12, 2025
Copy link
Owner Author

@connor-powers connor-powers left a comment

Choose a reason for hiding this comment

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

LGTM

@connor-powers connor-powers merged commit cf3d172 into main May 12, 2025
1 check passed
@connor-powers connor-powers deleted the dev branch May 12, 2025 04:34
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.

2 participants