Skip to content

Conversation

@jeff-hykin
Copy link
Member

@jeff-hykin jeff-hykin commented Jan 28, 2026

No description provided.

@jeff-hykin jeff-hykin changed the title RPC Rework: Part 6: add _connect_module_refs RPC Rework: With Var Renames Jan 29, 2026
@jeff-hykin jeff-hykin closed this Jan 29, 2026
@jeff-hykin jeff-hykin reopened this Jan 29, 2026
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

4 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@@ -0,0 +1,56 @@
# Copyright 2026 Dimensional Inc.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this example be in the docs? I don't think anyone will look at it here.

Copy link
Member Author

Choose a reason for hiding this comment

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

I should have updated blueprints.md for the docs and the had this as a standalone example for people who jump into the code. (I believe this dir is linked to somewhere in the docs)

paul-nechifor
paul-nechifor previously approved these changes Jan 29, 2026
return a * b

@rpc
def start(self) -> None: ...
Copy link
Contributor

Choose a reason for hiding this comment

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

You need super().start() here and stop in the others.

@paul-nechifor paul-nechifor merged commit 729ac9f into dev Jan 31, 2026
15 checks passed
@spomichter spomichter deleted the jeff/blueprint/6 branch February 1, 2026 03:33
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.

3 participants