gitea/tea
gitea/tea
57
129
Fork 64
Code Issues 97 Pull Requests 19 Actions Releases 16 Activity

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
Conversation 0 Commits 4 Files Changed 5 +4 -4
3 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 817631fbf2 - Show all commits

2
modules/interact/issue_create.go
View File

@ -16,7 +16,7 @@ func CreateIssue(login *config.Login, owner, repo string) error {
var title, description string
// owner, repo
owner, repo, err := PromptRepoSlug(owner, repo)
owner, repo, err := promptRepoSlug(owner, repo)
if err != nil {
return err
}

4
modules/interact/prompts.go
View File

@ -18,8 +18,8 @@ func PromptPassword(name string) (pass string, err error) {
return
}
// PromptRepoSlug interactively prompts for a Gitea repository or returns the current one
func PromptRepoSlug(defaultOwner, defaultRepo string) (owner, repo string, err error) {
// promptRepoSlug interactively prompts for a Gitea repository or returns the current one
func promptRepoSlug(defaultOwner, defaultRepo string) (owner, repo string, err error) {
YakoYakoYokuYoku marked this conversation as resolved Outdated
6543 commented 2020-12-14 19:14:12 +00:00
Outdated
Review
Copy Link

do we have to export it?

do we have to export it?
6543 commented 2020-12-14 19:15:02 +00:00
Outdated
Review
Copy Link

@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
YakoYakoYokuYoku commented 2020-12-14 19:25:00 +00:00
Outdated
Review
Copy Link

Gotcha

Gotcha
prompt := "Target repo:"
required := true
if len(defaultOwner) != 0 && len(defaultRepo) != 0 {

2
modules/interact/pull_create.go
View File

@ -17,7 +17,7 @@ func CreatePull(login *config.Login, owner, repo string) error {
var base, head, title, description string
// owner, repo
owner, repo, err := PromptRepoSlug(owner, repo)
owner, repo, err := promptRepoSlug(owner, repo)
if err != nil {
return err
}