temporalio / temporalio/sdk-python
[Feature Request] Make all exposed modules be packages
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 1.2k
- Forks
- 241
- Avg merge
- 3d 21h
- Merged PRs (30d)
- 55
Description
Describe the solution you'd like
temporalio.activity, temporalio.workflow, etc are all modules right now. Besides the obvious benefit of properly separating code in a package into multiple files, the other benefit is that we can control __all__ a bit more clearly so that users' intellisense doesn't display things we've imported (and in general they aren't able to access the imported things).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the exposed temporalio.activity and temporalio.workflow modules and reviewing how their public names are currently exposed. Done means the exposed modules are packages, can be split across multiple files, and provide clearer all behavior that does not present imported names as user-facing API.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- api
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100