precommits

[Fersoil]

solve precommit issues

workflows to include:

• ruff
• prettier

it is to decide whether will I use mypy, it could be useful, but produces unfixable errors

[Fersoil]

@WolodjaZ ruff reports errors if created any fixes. imho should inform user about the changes, but fix the code without throwing errors

[Fersoil]

Reopening again, since the issue is still not fixed. It seems that the problem is also with the pre-commit hooks, which are run before the ruff formatter.

I guess that the best solution is to remove the code formatting hooks from https://github.com/pre-commit/pre-commit-hooks.

[Fersoil]

Additional remark on pre-commits: It is somehow irritating that by default precommit fail if it makes any changes. Linked thread with this topic: https://stackoverflow.com/questions/58398995/black-as-pre-commit-hook-always-fails-my-commits

[WolodjaZ]

The authors mention that this is desired way of using recommits as developer always needs to check the changes that for example formatters suggest. But I agree that this kinda sucks in CI and I will think about best solution for this case.

[WolodjaZ]

Interesting discussion @Fersoil