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

Help command #63

Merged
merged 5 commits into from
Jun 21, 2024
Merged

Help command #63

merged 5 commits into from
Jun 21, 2024

Conversation

rin-st
Copy link
Member

@rin-st rin-st commented Jun 19, 2024

Issue: #8

Adds -h and --help flags. When one of these flags is added to cli input, help message will be shown and all other execution stops.

Help message:

image

Not sure about colors, everything white is also ok to me. But added them because there are other colors in our cli already

Is this enough for now? Or do I need to add something?

Copy link
Member

@damianmarti damianmarti left a comment

Choose a reason for hiding this comment

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

Working great!

And the text format looks good to me!

@technophile-04
Copy link
Collaborator

Ohh maybe we merge this after #58, since we are almost there? And then we could also add those options to help? Btw this looks really nice!

@rin-st
Copy link
Member Author

rin-st commented Jun 20, 2024

Ohh maybe we merge this after #58, since we are almost there? And then we could also add those options to help? Btw this looks really nice!

Thanks! Updated here.

I think we will also need to add information about curated extensions, solidity frameworks, probably link to readme. Or just add that information to readme + add link. I think we can do it in next iterations

@carletex
Copy link
Member

I think we will also need to add information about curated extensions

Yes! Will love this too. But yep, can be in future iterations. This looks good to me!

Copy link
Collaborator

@technophile-04 technophile-04 left a comment

Choose a reason for hiding this comment

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

Thanks @rin-st!! This is really nice!

@technophile-04 technophile-04 merged commit d3d4ce4 into main Jun 21, 2024
1 check passed
@technophile-04 technophile-04 deleted the help-command branch June 21, 2024 06:11
@technophile-04 technophile-04 mentioned this pull request Jun 21, 2024
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.

4 participants