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

Improve ockam tcp-inlet create --help text (clap command) #6643

Open
nazmulidris opened this issue Oct 18, 2023 · 5 comments · May be fixed by #8449
Open

Improve ockam tcp-inlet create --help text (clap command) #6643

nazmulidris opened this issue Oct 18, 2023 · 5 comments · May be fixed by #8449

Comments

@nazmulidris
Copy link
Contributor

Current behavior

When a user runs ockam tcp-inlet create --help they are presented with the following help information.

Image

Desired behavior

Instead, the help text should instead show the following: Create a TCP Inlet

Implementation details

You can change the Rust doc comment for the CreateCommand struct in this file to make this change.


We love helping new contributors! ❤️
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

@priyanshu9899
Copy link
Contributor

assign me this issue #6643 @nazmulidris. I would like to work on this .

@nazmulidris
Copy link
Contributor Author

@priyanshu9899 That's awesome, this is all yours. Please let us know if you have any questions as you explore. You can also ask questions on the contributors discord https://discord.gg/RAbjRr3kds

@ivor11
Copy link
Contributor

ivor11 commented Sep 8, 2024

Hi @nazmulidris , i'd like to submit a PR for this issue

@etorreborre
Copy link
Member

Hi @ivor11, please do.

@ivor11 ivor11 linked a pull request Sep 9, 2024 that will close this issue
5 tasks
@ivor11
Copy link
Contributor

ivor11 commented Sep 10, 2024

Hi @etorreborre , I've created a PR for this #8449

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants