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

chore: use ruff #1019

Merged
merged 3 commits into from
Jan 8, 2025
Merged

Conversation

jjaakola-aiven
Copy link
Contributor

About this change - What it does

References: #xxxxx

Why this way

Copy link

github-actions bot commented Jan 8, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  src/karapace
  auth.py 239
  client.py 79
  forward_client.py
  karapace_all.py 38
  rapu.py 295-299, 346, 401
  statsd.py 86
  utils.py 32, 200, 238
  src/karapace/avro_dataclasses
  introspect.py
  models.py
  src/karapace/backup
  api.py
  cli.py 168
  errors.py
  src/karapace/backup/backends/v3
  checksum.py
  src/karapace/compatibility/jsonschema
  utils.py
  src/karapace/coordinator
  schema_coordinator.py 586
  src/karapace/kafka
  common.py
  src/karapace/kafka_rest_apis
  __init__.py 102, 764
  consumer_manager.py 244, 260, 601
  src/karapace/protobuf
  io.py 187, 262, 268
  message_element.py
  proto_type.py
  src/schema_registry
  controller.py 816
  messaging.py 55
  reader.py 210, 228, 256, 327, 350
  src/schema_registry/middlewares
  __init__.py 30
Project Total  

The report is truncated to 25 files out of 29. To see the full report, please visit the workflow summary page.

This report was generated by python-coverage-comment-action

@jjaakola-aiven jjaakola-aiven marked this pull request as ready for review January 8, 2025 12:22
@jjaakola-aiven jjaakola-aiven requested review from a team as code owners January 8, 2025 12:22
@nosahama nosahama merged commit 5f6408b into jjaakola-aiven-fastapi Jan 8, 2025
10 checks passed
@nosahama nosahama deleted the jjaakola-aiven-fastapi-use-ruff branch January 8, 2025 14:35
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.

2 participants