ci: make reviewable

Signed-off-by: Julien Christophe <[email protected]>

Author: Sabotaz

README.md

@@ -60,7 +60,7 @@ Check the example:
 2. Create managed resources for your SQL server flavor:
 
    - **MySQL**: `Database`, `Grant`, `User` (See [the examples](examples/mysql))
-   - **PostgreSQL**: `Database`, `Grant`, `Extension`, `Role` (See [the examples](examples/postgresql))
+   - **PostgreSQL**: `Database`, `Schema`, `Grant`, `Extension`, `Role` (See [the examples](examples/postgresql))
    - **MSSQL**: `Database`, `Grant`, `User` (See [the examples](examples/mssql))
 
 [crossplane]: https://crossplane.io

apis/postgresql/v1alpha1/grant_types.go

@@ -18,12 +18,13 @@ package v1alpha1
 
 import (
 	"context"
-	"github.com/pkg/errors"
+
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"sigs.k8s.io/controller-runtime/pkg/client"
 
 	xpv1 "github.com/crossplane/crossplane-runtime/apis/common/v1"
 	"github.com/crossplane/crossplane-runtime/pkg/reference"
+	"github.com/pkg/errors"
 )
 
 const (
@@ -117,41 +118,41 @@ func (gp *GrantParameters) filledInFields() *stringSet {
 	return set
 }
 
+var grantTypeFields = map[GrantType][]string{
+	RoleMember:             {"MemberOf"},
+	RoleDatabase:           {"Database"},
+	RoleSchema:             {"Database", "Schema"},
+	RoleTable:              {"Database", "Schema", "Tables"},
+	RoleColumn:             {"Database", "Schema", "Tables", "Columns"},
+	RoleSequence:           {"Database", "Schema", "Sequences"},
+	RoleRoutine:            {"Database", "Schema", "Routines"},
+	RoleForeignServer:      {"Database", "ForeignServers"},
+	RoleForeignDataWrapper: {"Database", "ForeignDataWrappers"},
+}
+
 // IdentifyGrantType return the deduced GrantType from the filled in fields.
 func (gp *GrantParameters) IdentifyGrantType() (GrantType, error) {
-	fields := gp.filledInFields()
+	ff := gp.filledInFields()
 	pc := len(gp.Privileges)
 
-	grantTypeFields := map[GrantType][]string{
-		RoleMember:             {"MemberOf"},
-		RoleDatabase:           {"Database"},
-		RoleSchema:             {"Database", "Schema"},
-		RoleTable:              {"Database", "Schema", "Tables"},
-		RoleColumn:             {"Database", "Schema", "Tables", "Columns"},
-		RoleSequence:           {"Database", "Schema", "Sequences"},
-		RoleRoutine:            {"Database", "Schema", "Routines"},
-		RoleForeignServer:      {"Database", "ForeignServers"},
-		RoleForeignDataWrapper: {"Database", "ForeignDataWrappers"},
-	}
-
-	var role *GrantType
+	var gt *GrantType
 
-	for key, expectedFields := range grantTypeFields {
-		if fields.containsExactly(expectedFields...) {
-			role = &key
+	for k, v := range grantTypeFields {
+		if ff.containsExactly(v...) {
+			gt = &k
 			break
 		}
 	}
-	if role == nil {
+	if gt == nil {
 		return "", errors.New(errUnknownGrant)
 	}
-	if *role == RoleMember && pc > 0 {
+	if *gt == RoleMember && pc > 0 {
 		return "", errors.New(errMemberOfWithPrivileges)
 	}
-	if *role != RoleMember && pc < 1 {
+	if *gt != RoleMember && pc < 1 {
 		return "", errors.New(errNoPrivileges)
 	}
-	return *role, nil
+	return *gt, nil
 }
 
 // Some privileges are shorthands for multiple privileges. These translations