Skip to content

Add type hints to worker module#114

Open
kaypee90 wants to merge 2 commits into
mainfrom
add-worker-type-hints
Open

Add type hints to worker module#114
kaypee90 wants to merge 2 commits into
mainfrom
add-worker-type-hints

Conversation

@kaypee90

@kaypee90 kaypee90 commented Dec 1, 2025

Copy link
Copy Markdown
Owner

This PR adds comprehensive type hints to zebrok/worker.py, improving clarity and static analysis support.

Copilot AI left a comment

Copy link
Copy Markdown

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 adds type hints to the zebrok/worker.py module to improve code clarity and enable better static type checking. The changes focus on adding parameter and return type annotations to key methods in both the TaskQueueWorker and WorkerInitializer classes.

Key changes:

  • Added type hints to TaskQueueWorker.__init__ parameters
  • Added return type hint to _create_master_worker method
  • Added type hints to _initialize_slave_workers parameters for better clarity on expected types

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread zebrok/worker.py Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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