Update comments

Emyrk · Emyrk · commit 17ea3c30c3c6 · 2023-01-25T10:37:10.000-06:00
diff --git a/coderd/rbac/authz.go b/coderd/rbac/authz.go
@@ -27,7 +27,7 @@ import (
 // support unavailable to the user. Eg: Scope to a single resource.
 type ExpandableRoles interface {
 	Expand() ([]Role, error)
-	// Names is fo logging and tracing purposes, we want to know the human
+	// Names is for logging and tracing purposes, we want to know the human
 	// names of the expanded roles.
 	Names() []string
 }
diff --git a/coderd/rbac/builtin.go b/coderd/rbac/builtin.go
@@ -20,7 +20,8 @@ const (
 )
 
 // RoleNames is a list of user assignable role names. The role names must be
-// in the builtInRoles map.
+// in the builtInRoles map. Any non-user assignable roles will generate an
+// error on Expand.
 type RoleNames []string
 
 func (names RoleNames) Expand() ([]Role, error) {

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ import (`
`27`	`27`	`// support unavailable to the user. Eg: Scope to a single resource.`
`28`	`28`	`type ExpandableRoles interface {`
`29`	`29`	`Expand() ([]Role, error)`
`30`		`- // Names is fo logging and tracing purposes, we want to know the human`
	`30`	`+ // Names is for logging and tracing purposes, we want to know the human`
`31`	`31`	`// names of the expanded roles.`
`32`	`32`	`Names() []string`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,8 @@ const (`
`20`	`20`	`)`
`21`	`21`
`22`	`22`	`// RoleNames is a list of user assignable role names. The role names must be`
`23`		`-// in the builtInRoles map.`
	`23`	`+// in the builtInRoles map. Any non-user assignable roles will generate an`
	`24`	`+// error on Expand.`
`24`	`25`	`type RoleNames []string`
`25`	`26`
`26`	`27`	`func (names RoleNames) Expand() ([]Role, error) {`