diff options
| author | Jonathan Amsterdam <jba@google.com> | 2021-04-07 19:59:52 -0400 |
|---|---|---|
| committer | Jonathan Amsterdam <jba@google.com> | 2021-04-08 16:30:25 +0000 |
| commit | d189d9d19b2d6ec3358cc44d98c6354682aee052 (patch) | |
| tree | 4f8c04345729758b2de4be69f0643382a21ecccd /internal/postgres/unit.go | |
| parent | d5da038b60d9e6ee245c4cd69adad526aeb7d9b3 (diff) | |
| download | go-x-pkgsite-d189d9d19b2d6ec3358cc44d98c6354682aee052.tar.xz | |
internal/postgres: continue MustInsertModule cleanup
Remove calls to MustInsertModuleLMV.
It helped to reorganize TestGetLatestUnitVersion to be clearer about
the go.mod contents of the various module versions.
Change-Id: I7fa41bd742cf8bba2b5274172ad5b79fc22e73a9
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/308270
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Jonathan Amsterdam <jba@google.com>
Reviewed-by: Jamal Carvalho <jamal@golang.org>
Diffstat (limited to 'internal/postgres/unit.go')
| -rw-r--r-- | internal/postgres/unit.go | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/internal/postgres/unit.go b/internal/postgres/unit.go index ef41601e..5d48ebef 100644 --- a/internal/postgres/unit.go +++ b/internal/postgres/unit.go @@ -154,7 +154,6 @@ func (db *DB) getLatestUnitVersion(ctx context.Context, fullPath, requestedModul if requestedModulePath == internal.UnknownModulePath { modPaths = internal.CandidateModulePaths(fullPath) } - fmt.Printf("#### candidate paths: %v\n", modPaths) // Get latest-version information for all possible modules, from longest // to shortest path. lmvs, err := db.getMultiLatestModuleVersions(ctx, modPaths) @@ -162,7 +161,6 @@ func (db *DB) getLatestUnitVersion(ctx context.Context, fullPath, requestedModul return "", "", nil, err } for _, lmv = range lmvs { - fmt.Printf("#### lmv: %+v\n", lmv) // Collect all the versions of this module that contain fullPath. query := squirrel.Select("m.version"). From("modules m"). @@ -178,7 +176,6 @@ func (db *DB) getLatestUnitVersion(ctx context.Context, fullPath, requestedModul if err != nil { return "", "", nil, err } - fmt.Printf("#### allVersions = %v\n", allVersions) // Remove retracted versions. unretractedVersions := version.RemoveIf(allVersions, lmv.IsRetracted) // If there are no unretracted versions, move on. If we fall out of the @@ -195,7 +192,6 @@ func (db *DB) getLatestUnitVersion(ctx context.Context, fullPath, requestedModul unretractedVersions = version.RemoveIf(unretractedVersions, version.IsIncompatible) } latestVersion = version.LatestOf(unretractedVersions) - fmt.Printf("################ got latestVersion %q\n", latestVersion) break } if latestVersion != "" { |