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

Merged
6543 merged 10 commits from noerw/tea:issue-172-comments into master 4 months ago
noerw commented 4 months ago
  • 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 4 months ago
noerw added the
kind/feature
label 4 months ago
noerw added 2 commits 4 months ago
c965640040
show comments of PR
42887c9044
add `tea comment` to post a comment
noerw added the
status/needs-feedback
label 4 months ago
noerw removed the
status/needs-feedback
label 4 months ago
noerw added 2 commits 4 months ago
305d3381ba
add --comments flag, prompt only if necessary
1a0a3f802a
show comments for issues, add --comments flag
noerw added 1 commit 4 months ago
8db6b965c1
tea comment: print resulting comment
noerw changed title from WIP: add `tea comment` and show comments of issues/pulls to Add `tea comment` and show comments of issues/pulls 4 months ago
Poster

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

this is ready now (comment made with tea :D)
lunny approved these changes 4 months ago
Owner

Please resolve the conflicts.

Please resolve the conflicts.
noerw added 1 commit 4 months ago
b5e0ab2099
Merge branch 'master' into issue-172-comments
noerw added 1 commit 4 months ago
5953c36492
remove debug print statement
noerw added 1 commit 4 months ago
6543 added 1 commit 4 months ago
1a84c22b86
ret err
6543 approved these changes 4 months ago
noerw added 1 commit 4 months ago
019bd39176
fix lint
6543 merged commit 32b7b771cc into master 4 months ago
6543 deleted branch issue-172-comments 4 months ago

Reviewers

lunny approved these changes 4 months ago
6543 approved these changes 4 months ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as 32b7b771cc.
Sign in to join this conversation.
Loading…
There is no content yet.