Skip to content

[DOP-15650] Allow using Horizon with multiple uvicorn workers#60

Merged
dolfinus merged 1 commit intodevelopfrom
feature/DOP-15650
May 15, 2024
Merged

[DOP-15650] Allow using Horizon with multiple uvicorn workers#60
dolfinus merged 1 commit intodevelopfrom
feature/DOP-15650

Conversation

@dolfinus
Copy link
Member

@dolfinus dolfinus commented May 14, 2024

Change Summary

Few small adjustments to backend logging & usage examples to enable uvicorn workers.

Related issue number

Checklist

  • Commit message and PR title is comprehensive
  • Keep the change as small as possible
  • Unit and integration tests for the changes exist
  • Tests pass on CI and coverage does not decrease
  • Documentation reflects the changes where applicable
  • docs/changelog/next_release/<pull request or issue id>.<change type>.rst file added describing change
    (see CONTRIBUTING.rst for details.)
  • My PR is ready to review.

@dolfinus dolfinus self-assigned this May 14, 2024
@codecov
Copy link

codecov bot commented May 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.50%. Comparing base (4313940) to head (7eb2e43).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #60   +/-   ##
========================================
  Coverage    91.50%   91.50%           
========================================
  Files          120      120           
  Lines         2449     2449           
  Branches       240      240           
========================================
  Hits          2241     2241           
  Misses         162      162           
  Partials        46       46           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dolfinus dolfinus marked this pull request as ready for review May 14, 2024 12:35
@dolfinus dolfinus merged commit fd71fb8 into develop May 15, 2024
@dolfinus dolfinus deleted the feature/DOP-15650 branch May 15, 2024 07:52
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