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.

Sign up for GitHub

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

Jump to bottom

✨ Simplify system arguments #35

Merged
merged 2 commits into from
Apr 1, 2024
Merged

✨ Simplify system arguments #35

merged 2 commits into from
Apr 1, 2024

Conversation

GabrielePicco
Copy link
Contributor

✨ Simplify system arguments

Status Type ⚠️ Core Change Issue
Ready Feature No -

Problem

  • Arguments in a systems are always of type Vec<u8>, this allows to maintain a standard function signature and easily call from the central proxy program

Solution

  • System macro now allows to define serializable types and automatically parse and convert them

@GabrielePicco GabrielePicco merged commit 71c10f5 into main Apr 1, 2024
4 checks passed
@GabrielePicco GabrielePicco deleted the feat/improve-system-arguments branch April 1, 2024 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@GabrielePicco