Skip to content

refactoring: ♻️ improve naming consistency by renaming DeferredBus → DeferredDispatcher#50

Merged
remimd merged 1 commit intodevfrom
defer
Oct 12, 2025
Merged

refactoring: ♻️ improve naming consistency by renaming DeferredBus → DeferredDispatcher#50
remimd merged 1 commit intodevfrom
defer

Conversation

@remimd
Copy link
Member

@remimd remimd commented Oct 12, 2025

No description provided.

@remimd remimd requested a review from Copilot October 12, 2025 10:12
@remimd remimd self-assigned this Oct 12, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors the codebase to improve naming consistency by renaming DeferredBus to DeferredDispatcher and consolidating related interfaces.

  • Renames DeferredBus to DeferredDispatcher across the codebase
  • Moves DeferredDispatcher protocol definition from a standalone file to the base dispatcher module
  • Updates FastAPI integration to use the new naming convention and adds use_cache=False to dependency configuration

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
cq/ext/fastapi.pyi Removes old type definitions file
cq/ext/fastapi.py Updates FastAPI integration with new naming and adds proper type checking blocks
cq/_core/dispatcher/bus.py Reorders abstract methods in Bus protocol
cq/_core/dispatcher/base.py Adds DeferredDispatcher protocol definition
cq/_core/defer.py Removes old DeferredBus protocol file
cq/init.py Updates exports to use new DeferredDispatcher naming

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@remimd remimd merged commit 0148752 into dev Oct 12, 2025
6 checks passed
@remimd remimd deleted the defer branch October 12, 2025 10:16
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