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

feat: depend on cargo-witgen and inject args struct into wit subcommand #24

Merged
merged 13 commits into from
Apr 15, 2022

Conversation

willemneal
Copy link
Contributor

This prevents a duplicated implementation.

Also a general refactor to improve readability of CLI app.

This allows to not repeat the implementation of witgen.

Currently this doesn't support the contract methods.
chadoh added a commit to TENK-DAO/tenk that referenced this pull request Apr 13, 2022
chadoh added a commit to TENK-DAO/tenk that referenced this pull request Apr 13, 2022
chadoh added a commit to TENK-DAO/tenk that referenced this pull request Apr 13, 2022
chadoh added a commit to TENK-DAO/tenk that referenced this pull request Apr 13, 2022
@willemneal willemneal marked this pull request as ready for review April 14, 2022 20:38
willemneal pushed a commit to TENK-DAO/tenk that referenced this pull request Apr 14, 2022
* build: use experimental witme; speed up builds
Version from AhaLabs/witme#24

* fix: copy in standards code so that witme generates methods
Since witme no doesn't do a compile pass it can't expand the macros
Co-authored-by: Willem Wyndham <wyndham@cs.unc.edu>
@willemneal willemneal merged commit 4134f3a into main Apr 15, 2022
@willemneal willemneal deleted the feat/update_witgen branch April 15, 2022 20:30
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.

None yet

1 participant