Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: reduce event bus provider boilerplate code #391

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

links234
Copy link
Contributor

This is a pure refactor PR (no functional changes expected) to reduce boilerplate code for Storage Hub actors framework (event bus provider).

For more details please read README.md inside actors-derive crate.

@links234 links234 requested review from snowmead and ffarall March 11, 2025 13:21
@snowmead
Copy link
Contributor

We should have module/macro level documentation so that we could hover over the macros and see directly the documentation on how to use them. It is much simpler for developers.

Copy link
Contributor

@snowmead snowmead left a comment

Choose a reason for hiding this comment

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

Will approve when we add module/macro level documentation for ease of development

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