Add tea comment and show comments of issues/pulls #313

Merged
6543 merged 10 commits from noerw/tea:issue-172-comments into master 2020-12-21 16:07:36 +00:00
Member
  • Add tea comment <issue/pull index> [<body>] (partial fix to #172)
    • I like tea comment / tea c better than tea issue comment, but I'm open for suggestions
  • Add a prompt to tea pr <idx> if comments should be shown.
  • Add a prompt to tea issue <idx> if comments should be shown.
  • Add a --comments flag to avoid prompting (i.e. keep a programmatic mode).
    • if flag is unset, will prompt
    • if flag is set, will not prompt
    • if output is piped, will not prompt
- [x] Add `tea comment <issue/pull index> [<body>]` (partial fix to #172) - I like `tea comment` / `tea c` better than `tea issue comment`, but I'm open for suggestions - [x] Add a prompt to `tea pr <idx>` if comments should be shown. - [x] Add a prompt to `tea issue <idx>` if comments should be shown. - [x] Add a `--comments` flag to avoid prompting (i.e. keep a programmatic mode). - if flag is unset, will prompt - if flag is set, will not prompt - if output is piped, will not prompt
noerw added this to the v0.7.0 milestone 2020-12-17 15:16:22 +00:00
noerw added the
kind
feature
label 2020-12-17 15:16:22 +00:00
noerw added 2 commits 2020-12-17 15:16:22 +00:00
TODO: there needs to be a way to force running non-interactively
add tea comment to post a comment
All checks were successful
continuous-integration/drone/pr Build is passing
42887c9044
noerw added the
status/needs-feedback
label 2020-12-17 15:16:41 +00:00
noerw removed the
status/needs-feedback
label 2020-12-17 15:56:33 +00:00
noerw added 2 commits 2020-12-18 00:32:37 +00:00
don't prompt if --comments is provided, or output is piped
show comments for issues, add --comments flag
All checks were successful
continuous-integration/drone/pr Build is passing
1a0a3f802a
noerw added 1 commit 2020-12-18 00:38:12 +00:00
tea comment: print resulting comment
All checks were successful
continuous-integration/drone/pr Build is passing
8db6b965c1
noerw changed title from WIP: add `tea comment` and show comments of issues/pulls to Add `tea comment` and show comments of issues/pulls 2020-12-18 00:38:28 +00:00
Author
Member

this is ready now (comment made with tea :D)

this is ready now (comment made with tea :D)
lunny approved these changes 2020-12-18 04:10:40 +00:00
Dismissed
Owner

Please resolve the conflicts.

Please resolve the conflicts.
noerw added 1 commit 2020-12-21 13:45:32 +00:00
Merge branch 'master' into issue-172-comments
All checks were successful
continuous-integration/drone/pr Build is passing
b5e0ab2099
noerw added 1 commit 2020-12-21 14:48:50 +00:00
remove debug print statement
All checks were successful
continuous-integration/drone/pr Build is passing
5953c36492
unrelated, but better than opening another PR for this ;)
noerw added 1 commit 2020-12-21 15:44:07 +00:00
Merge remote-tracking branch 'upstream/master' into issue-172-comments
All checks were successful
continuous-integration/drone/pr Build is passing
5b64c6b0ba
6543 added 1 commit 2020-12-21 15:47:07 +00:00
ret err
Some checks failed
continuous-integration/drone/pr Build is failing
1a84c22b86
6543 approved these changes 2020-12-21 15:47:14 +00:00
Dismissed
noerw added 1 commit 2020-12-21 15:59:01 +00:00
fix lint
All checks were successful
continuous-integration/drone/pr Build is passing
019bd39176
6543 merged commit 32b7b771cc into master 2020-12-21 16:07:36 +00:00
6543 deleted branch issue-172-comments 2020-12-21 16:07:59 +00:00
Sign in to join this conversation.
No description provided.