Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

configure ruff format and run it on all remote modules #1426

Merged
merged 2 commits into from
Jun 20, 2024

Conversation

jluebbe
Copy link
Member

@jluebbe jluebbe commented Jun 20, 2024

This should make it easier to review the gRPC migration.

jluebbe added 2 commits June 20, 2024 09:32
Signed-off-by: Jan Luebbe <jlu@pengutronix.de>
This should make it easier to review the gRPC migration.

Signed-off-by: Jan Luebbe <jlu@pengutronix.de>
@jluebbe jluebbe added the grpc label Jun 20, 2024
@jluebbe jluebbe requested a review from Emantor June 20, 2024 07:47
@Emantor Emantor merged commit d790b52 into labgrid-project:master Jun 20, 2024
9 checks passed
Bastian-Krause added a commit to Bastian-Krause/labgrid that referenced this pull request Aug 13, 2024
…remote

The labgrid.remote module was formatted with ruff in labgrid-project#1426. The gRPC
changes are formatted correctly. Enforce formatting on the module now,
but exclude generated code.

We can enforce formatting of more modules over time.

Signed-off-by: Bastian Krause <bst@pengutronix.de>
Bastian-Krause added a commit to Bastian-Krause/labgrid that referenced this pull request Aug 13, 2024
…remote

The labgrid.remote module was formatted with ruff in labgrid-project#1426. The gRPC
changes are formatted correctly. Enforce formatting on the module now,
but exclude generated code.

We can enforce formatting of more modules over time.

Signed-off-by: Bastian Krause <bst@pengutronix.de>
Bastian-Krause added a commit to Bastian-Krause/labgrid that referenced this pull request Aug 19, 2024
…remote

The labgrid.remote module was formatted with ruff in labgrid-project#1426. The gRPC
changes are formatted correctly. Enforce formatting on the module now,
but exclude generated code.

We can enforce formatting of more modules over time.

Signed-off-by: Bastian Krause <bst@pengutronix.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants