diff options
| author | Amine Hilaly <hilalyamine@gmail.com> | 2019-05-25 17:27:25 +0200 |
|---|---|---|
| committer | Amine Hilaly <hilalyamine@gmail.com> | 2019-05-29 22:58:01 +0200 |
| commit | 43758a14fdd42ec9b25f1fe291ec774ed67cf34e (patch) | |
| tree | b26427fde633d82b069ee0e602df2125281fa2f7 /bridge | |
| parent | 0de2bd92b034ec265dbc66751626b4db67983760 (diff) | |
| download | git-bug-43758a14fdd42ec9b25f1fe291ec774ed67cf34e.tar.gz | |
Add warning messages for launchpad-preview
fix --target flag description
improve comments
Diffstat (limited to 'bridge')
| -rw-r--r-- | bridge/github/config.go | 6 | ||||
| -rw-r--r-- | bridge/launchpad/config.go | 7 |
2 files changed, 10 insertions, 3 deletions
diff --git a/bridge/github/config.go b/bridge/github/config.go index e723fda6..3baafadd 100644 --- a/bridge/github/config.go +++ b/bridge/github/config.go @@ -44,12 +44,12 @@ func (*Github) Configure(repo repository.RepoCommon, params core.BridgeParams) ( // getting owner and project name if params.Owner != "" && params.Project != "" { - // first try to use params if they are both provided + // first try to use params if both or project and owner are provided owner = params.Owner project = params.Project } else if params.URL != "" { - // try to parse them from params URL + // try to parse params URL and extract owner and project _, owner, project, err = splitURL(params.URL) if err != nil { return nil, err @@ -62,7 +62,7 @@ func (*Github) Configure(repo repository.RepoCommon, params core.BridgeParams) ( return nil, err } - // try terminal prompt + // terminal prompt owner, project, err = promptURL(remotes) if err != nil { return nil, err diff --git a/bridge/launchpad/config.go b/bridge/launchpad/config.go index 1c072ad3..6b299a9a 100644 --- a/bridge/launchpad/config.go +++ b/bridge/launchpad/config.go @@ -19,6 +19,13 @@ var ( ) func (*Launchpad) Configure(repo repository.RepoCommon, params core.BridgeParams) (core.Configuration, error) { + if params.Token != "" { + fmt.Println("warn: token is not needed to configure a launchpad-preview bridge") + } + if params.Owner != "" { + fmt.Println("warn: owner is not used when configuring a launchpad-preview bridge") + } + conf := make(core.Configuration) var err error var project string |