Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(141)

Unified Diff: logdog/appengine/coordinator/hierarchy/hierarchy.go

Issue 2626433004: Move "common/config" common types into cfgtypes. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: logdog/appengine/coordinator/hierarchy/hierarchy.go
diff --git a/logdog/appengine/coordinator/hierarchy/hierarchy.go b/logdog/appengine/coordinator/hierarchy/hierarchy.go
index 716ae5f886dc1b1ce2f90719bdd469400a7fd57f..e6b46c23e659f6962cea003368312fb23613316d 100644
--- a/logdog/appengine/coordinator/hierarchy/hierarchy.go
+++ b/logdog/appengine/coordinator/hierarchy/hierarchy.go
@@ -10,11 +10,11 @@ import (
"encoding/hex"
ds "github.com/luci/gae/service/datastore"
- "github.com/luci/luci-go/common/config"
log "github.com/luci/luci-go/common/logging"
"github.com/luci/luci-go/grpc/grpcutil"
"github.com/luci/luci-go/logdog/appengine/coordinator"
"github.com/luci/luci-go/logdog/common/types"
+ "github.com/luci/luci-go/luci_config/common/cfgtypes"
"golang.org/x/net/context"
"google.golang.org/grpc/codes"
@@ -169,7 +169,7 @@ type ListComponent struct {
type List struct {
// Project is the listed project name. If empty, the list refers to the
// project namespace.
- Project config.ProjectName
+ Project cfgtypes.ProjectName
// PathBase is the stream path base.
PathBase types.StreamPath
// Comp is the set of elements in this hierarchy result.
@@ -204,7 +204,7 @@ func Get(c context.Context, r Request) (*List, error) {
//
// We will validate our Project and PathBase types immediately afterwards.
l := List{
- Project: config.ProjectName(r.Project),
+ Project: cfgtypes.ProjectName(r.Project),
PathBase: types.StreamPath(r.PathBase),
}
« no previous file with comments | « logdog/appengine/coordinator/endpoints/services/service.go ('k') | logdog/appengine/coordinator/hierarchy/hierarchy_test.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698