Address comments

docs.md

@@ -251,7 +251,7 @@ This admission webhook prevents the disabling or deletion of a NodeDriver if the
 ClusterName must be equal to the namespace, and must refer to an existing `management.cattle.io/v3.Cluster` object. In addition, users cannot update the field after creation.
 
 #### BackingNamespace validation
-The BackingNamespace field cannot be changed once set. Projects without the BackingNamespace field can have it added.
+The `BackingNamespace` field cannot be changed once set. Projects without the `BackingNamespace` field can have it added.
 
 #### Protects system project
 
@@ -280,16 +280,16 @@ If `field.cattle.io/no-creator-rbac` annotation is set, `field.cattle.io/creator
 
 #### On create
 
-Populates the BackingNamespace field by concatenating `Project.ClusterName` and `Project.Name`.
+Populates the `BackingNamespace` field by concatenating `Project.ClusterName` and `Project.Name`.
 
 If the project is using a generated name (ie `GenerateName` is not empty), the generation happens within the mutating webhook.
-The reason for this is that the BackingNamespace is made up of the `Project.Name`, and name generation happens after mutating webhooks and before validating webhooks.
+The reason for this is that the `BackingNamespace` is made up of the `Project.Name`, and name generation happens after mutating and before validating webhooks.
 
 Adds the authz.management.cattle.io/creator-role-bindings annotation.
 
 #### On update
 
-If the BackingNamespace field is empty, populate the BackingNamespace field with the project name.
+If the `BackingNamespace` field is empty, it's populated with the project name.
 
 ## ProjectRoleTemplateBinding
 

pkg/resources/management.cattle.io/v3/project/Project.md

@@ -5,7 +5,7 @@
 ClusterName must be equal to the namespace, and must refer to an existing `management.cattle.io/v3.Cluster` object. In addition, users cannot update the field after creation.
 
 ### BackingNamespace validation
-The BackingNamespace field cannot be changed once set. Projects without the BackingNamespace field can have it added.
+The `BackingNamespace` field cannot be changed once set. Projects without the `BackingNamespace` field can have it added.
 
 ### Protects system project
 
@@ -34,13 +34,13 @@ If `field.cattle.io/no-creator-rbac` annotation is set, `field.cattle.io/creator
 
 ### On create
 
-Populates the BackingNamespace field by concatenating `Project.ClusterName` and `Project.Name`.
+Populates the `BackingNamespace` field by concatenating `Project.ClusterName` and `Project.Name`.
 
 If the project is using a generated name (ie `GenerateName` is not empty), the generation happens within the mutating webhook.
-The reason for this is that the BackingNamespace is made up of the `Project.Name`, and name generation happens after mutating webhooks and before validating webhooks.
+The reason for this is that the `BackingNamespace` is made up of the `Project.Name`, and name generation happens after mutating and before validating webhooks.
 
 Adds the authz.management.cattle.io/creator-role-bindings annotation.
 
 ### On update
 
-If the BackingNamespace field is empty, populate the BackingNamespace field with the project name.
+If the `BackingNamespace` field is empty, it's populated with the project name.

pkg/resources/management.cattle.io/v3/project/mutator.go

@@ -148,7 +148,7 @@ func (m *Mutator) createProjectNamespace(project *v3.Project) (*v3.Project, erro
 		backingNamespace = name.SafeConcatName(newProject.Spec.ClusterName, strings.ToLower(newProject.Name))
 		_, err = m.namespaceClient.Get(backingNamespace, v1.GetOptions{})
 		if err == nil {
-			return nil, fmt.Errorf("failed to create project: namespace %v already exists", backingNamespace)
+			return nil, fmt.Errorf("failed to create project: namespace %s already exists", backingNamespace)
 		} else if !apierrors.IsNotFound(err) {
 			return nil, err
 		}
@@ -158,9 +158,9 @@ func (m *Mutator) createProjectNamespace(project *v3.Project) (*v3.Project, erro
 	return newProject, nil
 }
 
-// updateProjectNamespace fills in BackingNamespace with the project name if it wasn't already set
-// this was the naming convention of project namespaces prior to using the BackingNamespace field. Filling
-// it here is just to maintain backwards compatibility
+// updateProjectNamespace fills in BackingNamespace with the project name if it wasn't already set.
+// This was the naming convention for project namespaces prior to using the BackingNamespace field.
+// Filling it here is just to maintain backwards compatibility.
 func (m *Mutator) updateProjectNamespace(project *v3.Project) *v3.Project {
 	if project.Status.BackingNamespace != "" {
 		return project