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

Sozo CLI --version displaying more info #1710

Merged
merged 4 commits into from
Mar 27, 2024

Conversation

fabrobles92
Copy link
Contributor

@fabrobles92 fabrobles92 commented Mar 27, 2024

Creation of fn generate_version that gathers versions of cairo, scarb, sierra and sozo and rewrite the --version default behavior given by library clap, looking now very similar to the output of scarb --version.

closes #1696
closes DOJ-290

Copy link
Collaborator

@glihm glihm left a comment

Choose a reason for hiding this comment

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

Great work, thank you @fabrobles92!

I've pushed minor fix in the testing that was causing CI to fail (not related to your code directly).

If it's your first contribution to dojo and you are OD eligible, please reach out to me on dojo discord.

@glihm glihm merged commit 3976f31 into dojoengine:main Mar 27, 2024
10 checks passed
@fabrobles92
Copy link
Contributor Author

@glihm Thank you very much for the guidance and the opportunity :)

kariy pushed a commit that referenced this pull request Mar 28, 2024
* Creation of fn generate_version that gathers versions of requested tools.

* fix: ensure saya test always wait tx to be executed

* fix: fmt

---------

Co-authored-by: glihm <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[sozo] output the cairo version when sozo --version is used
2 participants