Add interactive mode for `tea issue create` #302

Merged
6543 merged 4 commits from YakoYakoYokuYoku/tea:interactive-issue into master 4 months ago

Fixes #228.
Created #301 from my console.

Fixes #228. Created #301 from my console.
YakoYakoYokuYoku added 1 commit 4 months ago
8cb5aebf21
Implement interactive issue creation
noerw requested changes 4 months ago
noerw left a comment

great work! only a small nit

modules/utils/prompts.go Outdated
"github.com/AlecAivazis/survey/v2"
)
func PromptRepoSlug(defaultOwner, defaultRepo string) (owner, repo string, err error) {
noerw commented 4 months ago
Poster

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 4 months ago
5a34362591
Comment PromptRepoSlug
YakoYakoYokuYoku added 1 commit 4 months ago
1b59e8a45e
Move PromptRepoSlug to the right place
YakoYakoYokuYoku requested review from noerw 4 months ago
noerw added the
kind/feature
label 4 months ago
noerw added this to the v0.7.0 milestone 4 months ago
noerw approved these changes 4 months ago
khmarbaise approved these changes 4 months ago
6543 reviewed 4 months ago
modules/interact/prompts.go Outdated
}
// PromptRepoSlug interactively prompts for a Gitea repository or returns the current one
func PromptRepoSlug(defaultOwner, defaultRepo string) (owner, repo string, err error) {
6543 commented 4 months ago
Poster
Collaborator

do we have to export it?

do we have to export it?
6543 commented 4 months ago
Poster
Collaborator

@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

Gotcha

Gotcha
YakoYakoYokuYoku marked this conversation as resolved
YakoYakoYokuYoku added 1 commit 4 months ago
817631fbf2
Hide promptRepoSlug
6543 approved these changes 4 months ago
6543 merged commit b9f5ba0702 into master 4 months ago

Reviewers

noerw approved these changes 4 months ago
khmarbaise 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 b9f5ba0702.
Sign in to join this conversation.
Loading…
There is no content yet.