-
Notifications
You must be signed in to change notification settings - Fork 379
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
chore(gnodev): show link to docs upon 'h' key #2497
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2497 +/- ##
=======================================
Coverage 54.69% 54.69%
=======================================
Files 590 590
Lines 79036 79036
=======================================
Hits 43230 43230
Misses 32554 32554
Partials 3252 3252
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Updated, following recommandations: NoteIf we get shorter links after #2262 perhaps in the future this could be used: The red vertical shows the 80 cols and as you can see the line holds within the limit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
I propose we have "h" commandin gnodev mention a link to the full help of gnodev. This is a possible implementation. Tried a few variants, this one seems simple enough: ![ksnip_20240704-134249](https://github.com/gnolang/gno/assets/350354/f97bf22a-1028-45ad-a1b3-3f80b8b3d68f) @gfanton feel free to suggest changes :) --------- Co-authored-by: grepsuzette <grepsuzette@users.noreply.github.com> Co-authored-by: Leon Hudak <33522493+leohhhn@users.noreply.github.com>
I propose we have "h" commandin gnodev mention a link to the full help of gnodev. This is a possible implementation.
Tried a few variants, this one seems simple enough:
@gfanton feel free to suggest changes :)