Add ruff format check to ci.yml

[km-64]

Added format checking with ruff using ruff format --check.
See: #2944 (comment)

[janiversen]

LGTM, thanks

[janiversen]

Something seems a bit off, "ruff format --check" runs ok on dev, but this PR have an error

[km-64]

Looks like test/client/test_client.py is not formatted. @janiversen It looks like you formatted it recently. Maybe you're using a different version of ruff than I and the ci are using? I'm on ruff 0.15.18.

[km-64]

When I run ruff format it wants to make these changes.

diff --git a/test/client/test_client.py b/test/client/test_client.py
index 7e9f5a4b..ee052c3c 100755
--- a/test/client/test_client.py
+++ b/test/client/test_client.py
@@ -380,7 +380,8 @@ class TestMixin:
 
         with pytest.raises(TypeError):
             ModbusClientMixin.convert_to_registers(
-                bool, ModbusClientMixin.DATATYPE.BITS  # type: ignore[arg-type]
+                bool,
+                ModbusClientMixin.DATATYPE.BITS,  # type: ignore[arg-type]
             )
 
     def test_client_mixin_convert_datatype_fail(self):

[janiversen]

Looks correct, feel free to submit it as a separate PR.

I expect that Zuban will mark an error.

[janiversen]

Please do not mix code changes with CI changes.

[km-64]

The problem was that ruff moved the ignore comment that zuban relied on. I formatted with ruff and fixed teh ignore comment in #2948.

[km-64]

#2948 could be merged first. Then I can then merge those changes into this PR so that all the new ci checks pass. I'll tidy up this pr to only include changes to ci.yml.

[janiversen]

Good catch (check_ci).

LGTM, thanks

[janiversen]

Lets get it merged.