Skip to content

Cosmetic code changes - #329

Open
adrianusler wants to merge 4 commits into
ECSHackWeek:mainfrom
adrianusler:maint/cosmetic-code-changes
Open

Cosmetic code changes#329
adrianusler wants to merge 4 commits into
ECSHackWeek:mainfrom
adrianusler:maint/cosmetic-code-changes

Conversation

@adrianusler

Copy link
Copy Markdown
Contributor

Just a few cosmetic changes that I think help with code readability. Functionality is unaffected by these changes. If there should be clashes with conventions agreed upon in this project, please let me know!

@coveralls

coveralls commented Mar 30, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 32706319019

Coverage remained the same at 99.883%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 55 of 55 lines across 2 files are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 852
Covered Lines: 851
Line Coverage: 99.88%
Coverage Strength: 1.0 hits per line

💛 - Coveralls

@adrianusler

Copy link
Copy Markdown
Contributor Author

This is ready to merge, @mdmurbach @BGerwe @nealde.
The coveralls check only fails because the overall number of code line decreases with this Pull Request. There are no untested lines in this Pull Request.

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