Skip to content

Conversation

@TimmRuppert
Copy link
Contributor

Very minor code cosmetics like:

  • f-strings without any placeholders
  • unused imports

@pmai
Copy link
Contributor

pmai commented Jul 1, 2025

Since they mostly apply to non-osirules checker code, I'd retarget that to main and handle separately - the osirules relevant two changes are now incorporated in the osirules PR.

@TimmRuppert TimmRuppert changed the base branch from feature/automatic-checks to main July 1, 2025 11:59
@TimmRuppert
Copy link
Contributor Author

Since they mostly apply to non-osirules checker code, I'd retarget that to main and handle separately - the osirules relevant two changes are now incorporated in the osirules PR.

I just re-targeted to main and removed changes related to #6

Signed-off-by: Timm Ruppert <timm.ruppert@persival.de>
@pmai pmai force-pushed the code_cosmetics branch from c1c67cc to 8a63a5c Compare July 8, 2025 12:56
@pmai pmai merged commit 3c850d0 into OpenSimulationInterface:main Jul 8, 2025
8 checks passed
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.

3 participants