Steven Rieder

Results 306 comments of Steven Rieder

> Nice, but we should probably pick one tool and not run two, as they may end up fighting each other. Also, possibly [ruff](https://docs.astral.sh/ruff/) should be under consideration too? yes...

@LourensVeen I'm not sure how to test the auto-fix part of this, do you have any ideas for that?

I've tested it on a different repository now. The linter will push a fix to the branch in the PR, which seems fine to me.

I'm just not sure yet if this will fix only the files affected by the PR (which should be how this works) or if it checks all other files too...

Not ready for merging before the following is fixed: - [ ] don't write files to disk in addition to reporting via AMUSE - [ ] ensure we consistently report...

@cournoyercloutierc have you been able to try this?

This issue is closely related to #1123

> In addition: Some text missing here I guess? It would be good to harmonize function/method names more, I agree.

I think that if a function does *exactly* the same, the name of that function should also be the same (facilitating switching codes). If it does something *nearly* the same,...