Skip to content

chore: rearrange bindings and entrypoints classes#150

Open
ryanking13 wants to merge 1 commit into
mainfrom
gyeongjae/refactor-bindings
Open

chore: rearrange bindings and entrypoints classes#150
ryanking13 wants to merge 1 commit into
mainfrom
gyeongjae/refactor-bindings

Conversation

@ryanking13

Copy link
Copy Markdown
Contributor

This is my last code refactor PR:

  • Remove workers/_worker.py
  • Move bindings wrapper implementation to rpc.py
  • Move entry points (DurableObjects, Workflows) to a separate entrypoint.py file

@ryanking13 ryanking13 changed the title chore: refactor bindings and entrypoints chore: rearrange bindings and entrypoints classes Jul 7, 2026
@ask-bonk

ask-bonk Bot commented Jul 7, 2026

Copy link
Copy Markdown

The refactor is clean. All code has been properly moved, all imports updated, the circular import problem is solved by co-locating _BindingWrapper with python_to_rpc/python_from_rpc in rpc.py, and all tests reference the new paths. No code was lost, no exports are missing, and no stale references remain.

LGTM

github run

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.

1 participant