Add interactive mode for tea issue create #302

Merged
6543 merged 4 commits from YakoYakoYokuYoku/tea:interactive-issue into master 2020-12-14 20:05:32 +00:00
Contributor

Fixes #228.
Created #301 from my console.

Fixes #228. Created #301 from my console.
YakoYakoYokuYoku added 1 commit 2020-12-14 18:16:25 +00:00
Implement interactive issue creation
Some checks failed
continuous-integration/drone/pr Build is failing
8cb5aebf21
Signed-off-by: Martin Reboredo <yakoyoku@gmail.com>
noerw requested changes 2020-12-14 18:24:55 +00:00
Dismissed
noerw left a comment
Member

great work! only a small nit

great work! only a small nit
@ -0,0 +11,4 @@
"github.com/AlecAivazis/survey/v2"
)
func PromptRepoSlug(defaultOwner, defaultRepo string) (owner, repo string, err error) {
Member

I'd put this into modules/interact/prompts.go

I'd put this into `modules/interact/prompts.go`
YakoYakoYokuYoku marked this conversation as resolved
YakoYakoYokuYoku added 1 commit 2020-12-14 18:26:55 +00:00
Comment PromptRepoSlug
All checks were successful
continuous-integration/drone/pr Build is passing
5a34362591
YakoYakoYokuYoku added 1 commit 2020-12-14 18:34:48 +00:00
Move PromptRepoSlug to the right place
All checks were successful
continuous-integration/drone/pr Build is passing
1b59e8a45e
YakoYakoYokuYoku requested review from noerw 2020-12-14 18:37:34 +00:00
noerw added the
kind
feature
label 2020-12-14 18:37:40 +00:00
noerw added this to the v0.7.0 milestone 2020-12-14 18:37:43 +00:00
noerw approved these changes 2020-12-14 18:38:10 +00:00
Dismissed
khmarbaise approved these changes 2020-12-14 18:49:15 +00:00
Dismissed
6543 reviewed 2020-12-14 19:14:12 +00:00
Dismissed
@ -16,1 +19,4 @@
}
// PromptRepoSlug interactively prompts for a Gitea repository or returns the current one
func PromptRepoSlug(defaultOwner, defaultRepo string) (owner, repo string, err error) {
Owner

do we have to export it?

do we have to export it?
Owner

@YakoYakoYokuYoku nice work, just one nit

exporting this function is not needed if it is only used in the same module

@YakoYakoYokuYoku nice work, just one nit exporting this function is not needed if it is only used in the same module
Author
Contributor

Gotcha

Gotcha
YakoYakoYokuYoku marked this conversation as resolved
YakoYakoYokuYoku added 1 commit 2020-12-14 19:27:40 +00:00
Hide promptRepoSlug
All checks were successful
continuous-integration/drone/pr Build is passing
817631fbf2
6543 approved these changes 2020-12-14 20:05:03 +00:00
Dismissed
6543 merged commit b9f5ba0702 into master 2020-12-14 20:05:32 +00:00
Sign in to join this conversation.
No description provided.