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

[adapter] adapter CLI "run" command needs to support running module functions #137

Closed
lxfind opened this issue Jan 7, 2022 · 1 comment
Closed

[adapter] adapter CLI "run" command needs to support running module functions #137

lxfind opened this issue Jan 7, 2022 · 1 comment

Comments

@lxfind
Copy link
Contributor

lxfind commented Jan 7, 2022

Current when we use adapter CLI with the "run" command (fastx sandbox run ...), it goes through Move script run path. This doesn't work for us. We want to run module functions instead of scripts, so that the signature allows us to pass objects such as TxContext.
@lxfind lxfind mentioned this issue Jan 7, 2022
Merged
@lxfind lxfind changed the title adapter CLI "run" command needs to support running module functions [adapter] adapter CLI "run" command needs to support running module functions Jan 21, 2022
@lxfind lxfind added this to the TBD milestone Jan 21, 2022
@lxfind
Copy link
Contributor Author

lxfind commented Jan 25, 2022

Adapter CLI was deprecated.

@lxfind lxfind closed this as completed Jan 25, 2022
mwtian pushed a commit that referenced this issue Sep 12, 2022
@Clay-Mysten
Update README.md (#137)
7ac7878 
Add links to Fabric and fabfile.py from Narwhal README
mwtian pushed a commit to mwtian/sui that referenced this issue Sep 29, 2022
@Clay-Mysten
Update README.md (MystenLabs#137)
d0f04f1 
Add links to Fabric and fabfile.py from Narwhal README
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lxfind