Skip to content

Add linting to python run#759

Merged
tconley1428 merged 13 commits intomainfrom
python_lint
Feb 26, 2026
Merged

Add linting to python run#759
tconley1428 merged 13 commits intomainfrom
python_lint

Conversation

@tconley1428
Copy link
Contributor

What was changed

Why?

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

@tconley1428 tconley1428 requested a review from a team as a code owner February 18, 2026 20:01
@Sushisource
Copy link
Member

Looks like we need to add poe to the docker file tho

@tconley1428 tconley1428 requested a review from a team as a code owner February 19, 2026 18:12
tconley1428 and others added 11 commits February 19, 2026 10:13
…cate module errors

This creates __init__.py files in all directories under features/, not just
the leaf directories. This ensures proper Python package hierarchy so mypy
sees fully qualified module names (e.g., features.update.basic vs
features.signal.basic) instead of conflicting module names.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Changed mypy command to scan ../features instead of ../ to prevent
conflicts between harness/python/feature.py being seen as both "feature"
and "harness.python.feature" modules.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- signal/basic: Use Workflow.run instead of Workflow class for start_workflow
- continue_as_new: Use Workflow.run and fix argument positioning
- cancel_try_cancel: Use get_workflow_handle instead of get_workflow_handle_for

These fixes address the type checking errors found by the newly integrated
mypy linting in the build process.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@tconley1428 tconley1428 merged commit d59ab87 into main Feb 26, 2026
26 checks passed
@tconley1428 tconley1428 deleted the python_lint branch February 26, 2026 18:09
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