diff --git a/doRelease.go b/doRelease.go index 4a8c42f..91b3449 100644 --- a/doRelease.go +++ b/doRelease.go @@ -62,7 +62,7 @@ func doRelease() bool { var all [][]string all = append(all, []string{"git", "add", "-f", "go.mod"}) - if me.current.Status.IsPrimitive() { + if ok, _ := check.IsPrimitive(); ok { // don't add go.sum here. TODO: check for go.sum file and fail } else { all = append(all, []string{"git", "add", "-f", "go.sum"}) diff --git a/findNext.go b/findNext.go index 4c368e9..ce1fd6d 100644 --- a/findNext.go +++ b/findNext.go @@ -4,6 +4,7 @@ package main import ( "go.wit.com/log" + "go.wit.com/lib/gui/repolist" "go.wit.com/lib/protobuf/gitpb" ) @@ -24,11 +25,8 @@ func findNext() bool { log.Info("boo, you didn't git clone", repo.GoPath()) return false } - log.Info("findNext() STARTING", check.GoPath) - log.Info("findNext() STARTING", check.GoPath) - log.Info("findNext() STARTING", check.GoPath) if check.GetTargetVersion() == check.GetCurrentBranchVersion() { - log.Info("findNext() no update needed", check.GoPath, check.GetTargetVersion(), "vs", check.GetCurrentBranchVersion()) + // log.Info("findNext() no update needed", check.GoPath, check.GetTargetVersion(), "vs", check.GetCurrentBranchVersion()) continue } else { log.Info("findNext() update needed", check.GoPath, check.GetTargetVersion(), "vs", check.GetCurrentBranchVersion()) @@ -126,3 +124,21 @@ func fixGodeps(check *gitpb.Repo) bool { } return good } + +func setCurrentRepo(repo *repolist.RepoRow, s string, note string) bool { + check := me.forge.Repos.FindByGoPath(repo.GoPath()) + if check == nil { + log.Info("boo, you didn't git clone", repo.GoPath()) + return false + } + + me.release.repo.SetText(check.GoPath) + me.release.status.SetText(s) + me.release.notes.SetText(note) + me.current = repo + me.release.version.SetText(check.GetTargetVersion()) + me.release.releaseVersionB.SetText("release version " + check.GetTargetVersion()) + me.release.openrepo.Enable() + + return true +} diff --git a/http.go b/http.go index 99b2863..846f93a 100644 --- a/http.go +++ b/http.go @@ -259,22 +259,28 @@ func showNext() { return } + check := me.forge.Repos.FindByGoPath(me.current.GoPath()) + if check == nil { + log.Info("forge: how was this not found?", me.current.GoPath()) + return + } + if me.current.Status.IsReleased() { - log.Info("IsReleased() == true SHOULD NOT RELEASE THIS") + log.Info("IsReleased() == true. do not release this a second time") } else { log.Info("IsReleased() == false") } - if me.current.Status.CheckDirty() { - log.Info("CheckDirty() == true. SHOULD NOT RELEASE THIS") + if check.CheckDirty() { + log.Info("CheckDirty() == true. do not release dirty repos") } else { log.Info("CheckDirty() == false") } - if me.current.Status.IsPrimitive() { + if ok, _ := check.IsPrimitive(); ok { log.Info("IsPrimitive() == true") } else { log.Info("IsPrimitive() == false") } - if me.current.Status.IsPrivate() { + if me.forge.IsPrivate(check.GoPath) { log.Info("IsPrivate() == true") } else { log.Info("IsPrivate() == false") diff --git a/human.go b/human.go index fab7e94..e771238 100644 --- a/human.go +++ b/human.go @@ -74,7 +74,11 @@ func PrintReleaseReport(readonly string, perfect string) { log.Info("boo, you didn't git clone", check.GoPath) continue } - log.Info(me.forge.StandardReleaseHeader(check, "")) + var state string + if check.CheckDirty() { + state = "(dirty)" + } + log.Info(me.forge.StandardReleaseHeader(check, state)) } log.Info(fmt.Sprintf("total repo count = %d", count)) } diff --git a/releaseBox.go b/releaseBox.go index 99f50c7..d366f63 100644 --- a/releaseBox.go +++ b/releaseBox.go @@ -238,19 +238,3 @@ func buttonDisable() { func buttonEnable() { me.Enable() } - -func setCurrentRepo(newcur *repolist.RepoRow, s string, note string) bool { - if newcur.ReadOnly() { - return false - } - - me.release.repo.SetText(newcur.GoPath()) - me.release.status.SetText(s) - me.release.notes.SetText(note) - me.current = newcur - me.release.version.SetText(me.current.Status.GetTargetVersion()) - me.release.releaseVersionB.SetText("release version " + me.current.Status.GetTargetVersion()) - me.release.openrepo.Enable() - - return true -}