Skip to content

Rework grpc locations and add some different structure#3

Merged
cretz merged 6 commits intotemporalio:mainfrom
cretz:structure1
Jan 31, 2022
Merged

Rework grpc locations and add some different structure#3
cretz merged 6 commits intotemporalio:mainfrom
cretz:structure1

Conversation

@cretz
Copy link
Copy Markdown
Contributor

@cretz cretz commented Jan 31, 2022

What was changed

  • temporalio/proto/temporal/api -> temporalio/api
  • temporalio/proto/dependencies -> temporalio/api/dependencies
  • temporalio/proto/temporal/sdk -> temporalio/bridge/proto
  • Add aliases for all protos/grpc to top-level of respective module for clarity
  • Add LICENSE
  • Add simple pure-python gRPC test
  • Add isort to format/lint task
  • Add mypy lint check and mypy-protobuf support
  • Add pytest task

(carryover from #2)

@cretz cretz requested review from Sushisource and bergundy January 31, 2022 17:36
@bergundy
Copy link
Copy Markdown
Member

Already reviewed in #2

@cretz cretz merged commit a3d5328 into temporalio:main Jan 31, 2022
@cretz cretz deleted the structure1 branch March 18, 2022 14:33
mfateev added a commit to mfateev/sdk-python that referenced this pull request Dec 27, 2025
- Convert Pydantic models to dataclasses in _models.py (item temporalio#3)
- Standardize type annotations: replace Optional[X] with X | None (item temporalio#9)
- Verify docstring style follows SDK conventions (item temporalio#4)
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