gitea/tea
gitea/tea
58
136
Fork 67
Code Issues 104 Pull Requests 8 Actions Releases 16 Activity

Proper help text & new README structure #311

Merged
lunny merged 11 commits from noerw/tea:improve-app-help into master 2020-12-21 13:37:21 +00:00
Conversation 0 Commits 11 Files Changed 17 +172 -60
noerw commented 2020-12-17 15:03:02 +00:00
Member
Copy Link
  • rework the help text output of tea, tea help, tea --help, now including a proper description, examples, about, ..
  • group commands by category in the help text
  • add a new (current) demo gif in README
  • replace the Usage section of the README with raw tea help output: IMO having a different text in the readme is a lame excuse to not ship good help in the binary.
  • remotely related: make tea --version output only the version, for enhanced parseability (breaking?)

let me know what you think

preview the new readme here

- rework the help text output of `tea`, `tea help`, `tea --help`, now including a proper description, examples, about, .. - group commands by category in the help text - add a new (current) demo gif in README - replace the `Usage` section of the README with raw `tea help` output: IMO having a different text in the readme is a lame excuse to not ship good help in the binary. - remotely related: make `tea --version` output **only** the version, for enhanced parseability (breaking?) let me know what you think preview the new readme [here](https://gitea.com/noerw/tea/src/branch/improve-app-help)
noerw added the
kind/docs
 label 2020-12-17 15:03:02 +00:00
noerw added 8 commits 2020-12-17 15:03:02 +00:00 
It's an antipattern to have different help texts aimed at the same
users. So now that we have a good cli help text, lets use it here.
This eases maintenance, and at the same time gives an honest impression
on what we have to offer, while also encouraging to improve the internal
help text in the future.

I feel a bit sad for the GIF, but it was becoming outdated anyway..
shows the (probably) most useful workflow
Merge branch 'master' into improve-app-help
All checks were successful
continuous-integration/drone/pr Build is passing
Details
0e2ba56ab5
6543 Comment%!(EXTRA template.HTML=2020-12-17 17:36:44 +00:00)
Dismissed
README.md Outdated
@ -6,3 +6,2 @@
Tea is a command line tool for interacting on one or more Gitea instances.
It uses [code.gitea.io/sdk](https://code.gitea.io/sdk) and interacts with the Gitea API
![demo gif](./demo.gif)
6543 commented 2020-12-17 17:36:44 +00:00
Owner
Copy Link

@techknowlogick can you update demo gif on dl.gitea.io with the gif of this pull?

@techknowlogick can you update demo gif on dl.gitea.io with the gif of this pull?
6543 Comment%!(EXTRA template.HTML=2020-12-17 17:37:52 +00:00)
Dismissed
README.md Outdated
@ -10,1 +9,3 @@
![demo gif](https://dl.gitea.io/screenshots/tea_demo.gif)
```
tea - command line tool to interact with Gitea
version 0.6.0+17-g1c10f33
6543 commented 2020-12-17 17:37:52 +00:00
Owner
Copy Link

version 0.7.0 preview ... or so (should be changed on release)

version 0.7.0 preview ... or so (should be changed on release)
noerw marked this conversation as resolved
6543 Comment%!(EXTRA template.HTML=2020-12-17 17:40:43 +00:00)
Dismissed
main.go Outdated
@ -60,1 +67,4 @@
}
var appDescription = `tea is a productivity helper for Gitea. It can be used to manage most entities on one
or multiple Gitea instances, and also provides local helpers like 'tea pull checkout'.
6543 commented 2020-12-17 17:40:43 +00:00
Owner
Copy Link
-or multiple Gitea instances, and also provides local helpers like 'tea pull checkout'.
+or multiple Gitea instances and provides local helpers like 'tea pull checkout'.
```diff -or multiple Gitea instances, and also provides local helpers like 'tea pull checkout'. +or multiple Gitea instances and provides local helpers like 'tea pull checkout'.
noerw marked this conversation as resolved
noerw added 1 commit 2020-12-17 19:13:32 +00:00
code review
All checks were successful
continuous-integration/drone/pr Build is passing
Details
78c94edaae
noerw added 1 commit 2020-12-17 19:24:57 +00:00
Merge branch 'master' into improve-app-help
All checks were successful
continuous-integration/drone/pr Build is passing
Details
d9124476b7
lunny Comment%!(EXTRA template.HTML=2020-12-18 04:14:56 +00:00)
Dismissed
README.md Outdated
@ -29,41 +91,6 @@ brew tap gitea/tap https://gitea.com/gitea/homebrew-gitea
brew install tea
lunny commented 2020-12-18 04:14:56 +00:00
Owner
Copy Link

I'd like to add ### macOS before these lines and also move them before alphinelinux since this is an official one.

I'd like to add `### macOS` before these lines and also move them before alphinelinux since this is an official one.
noerw marked this conversation as resolved
lunny approved these changes 2020-12-18 04:17:00 +00:00
Dismissed
lunny left a comment
Owner
Copy Link

Otherwise LGTM

Otherwise LGTM
khmarbaise approved these changes 2020-12-18 06:53:49 +00:00
Dismissed
README.md Outdated
@ -4,3 +4,3 @@
## The official CLI interface for gitea
### the official CLI for gitea
khmarbaise commented 2020-12-18 06:51:49 +00:00
Member
Copy Link

I would suggest to use ### The official CLI for gitea instead.

I would suggest to use `### The official CLI for gitea` instead.
noerw marked this conversation as resolved
noerw added 1 commit 2020-12-18 07:36:18 +00:00
restructure installation section
All checks were successful
continuous-integration/drone/pr Build is passing
Details
2c6c4dbb0a
lunny merged commit 43a58bdba1 into master 2020-12-21 13:37:21 +00:00
noerw deleted branch improve-app-help 2020-12-21 13:40:50 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
kind/breaking

   
kind/bug

   
kind/build

   
kind/dependency

   
kind/deployment

   
kind/docs
  
kind
enhancement

  
kind
feature

  
kind/proposal
  
kind
question

  
kind
refactor

  
kind/security

   
kind/testing

   
kind/translation

   
priority/critical

   
priority/high

   
priority/low

   
priority/medium

   
reviewed/duplicate

   
reviewed/invalid

   
reviewed/wontfix

   
skip-changelog

   
status/blocked

   
status/has-backport

   
status/has-pull

   
status/needs-backport

   
status/needs-feedback

   
status/needs-reviews

   
status/wip

   
upstream/gitea

   
upstream/sdk
No Label
kind/breaking
kind/bug
kind/build
kind/dependency
kind/deployment
kind/docs
kind
enhancement
kind
feature
kind/proposal
kind
question
kind
refactor
kind/security
kind/testing
kind/translation
priority/critical
priority/high
priority/low
priority/medium
reviewed/duplicate
reviewed/invalid
reviewed/wontfix
skip-changelog
status/blocked
status/has-backport
status/has-pull
status/needs-backport
status/needs-feedback
status/needs-reviews
status/wip
upstream/gitea
upstream/sdk
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: gitea/tea#311
No description provided.
Delete Branch "noerw/tea:improve-app-help"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?