Removed some more stale TODO assignments and standardized on TODO instead of NOTE (#14644) (#23722)

[upstream:869cda327d7e21196577aed97c31142abb0c7130]

Signed-off-by: Modular Magician <[email protected]>

Author: modular-magician

google/services/resourcemanager/resource_google_project.go

@@ -790,7 +790,7 @@ func doEnableServicesRequest(services []string, project, billingProject, userAge
 // Handle errors that are retryable at call time for serviceusage
 // Specifically, errors in https://cloud.google.com/service-usage/docs/reference/rest/v1/services/batchEnable#response-body
 // Errors in operations are handled separately.
-// NOTE(rileykarson): This should probably be turned into a retry predicate
+// TODO: This should probably be turned into a retry predicate
 func handleServiceUsageRetryablePreconditionError(err error) error {
 	if err == nil {
 		return nil

google/services/servicenetworking/resource_google_service_networking_peered_dns_domain.go

@@ -256,7 +256,7 @@ func resourceGoogleServiceNetworkingPeeredDNSDomainDelete(d *schema.ResourceData
 	return nil
 }
 
-// NOTE(deviavir): An out of band aspect of this API is that it uses a unique formatting of network
+// TODO: An out of band aspect of this API is that it uses a unique formatting of network
 // different from the standard self_link URI. It requires a call to the resource manager to get the project
 // number for the current project.
 func getProjectNumber(d *schema.ResourceData, config *transport_tpg.Config, project, userAgent string) (string, error) {

google/services/servicenetworking/resource_service_networking_connection.go

@@ -57,7 +57,7 @@ func ResourceServiceNetworkingConnection() *schema.Resource {
 				DiffSuppressFunc: tpgresource.CompareSelfLinkOrResourceName,
 				Description:      `Name of VPC network connected with service producers using VPC peering.`,
 			},
-			// NOTE(craigatgoogle): This field is weird, it's required to make the Insert/List calls as a parameter
+			// TODO: This field is weird, it's required to make the Insert/List calls as a parameter
 			// named "parent", however it's also defined in the response as an output field called "peering", which
 			// uses "-" as a delimiter instead of ".". To alleviate user confusion I've opted to model the gcloud
 			// CLI's approach, calling the field "service" and accepting the same format as the CLI with the "."
@@ -357,7 +357,7 @@ func resourceServiceNetworkingConnectionImportState(d *schema.ResourceData, meta
 	return []*schema.ResourceData{d}, nil
 }
 
-// NOTE(craigatgoogle): The Connection resource in this API doesn't have an Id field, so inorder
+// TODO: The Connection resource in this API doesn't have an Id field, so inorder
 // to support the Read method, we create an Id using the tuple(Network, Service).
 type connectionId struct {
 	Network string
@@ -395,7 +395,7 @@ func parseConnectionId(id string) (*connectionId, error) {
 	}, nil
 }
 
-// NOTE(craigatgoogle): An out of band aspect of this API is that it uses a unique formatting of network
+// TODO: An out of band aspect of this API is that it uses a unique formatting of network
 // different from the standard self_link URI. It requires a call to the resource manager to get the project
 // number for the current project.
 func RetrieveServiceNetworkingNetworkName(d *schema.ResourceData, config *transport_tpg.Config, network, userAgent string) (string, error) {
@@ -438,7 +438,7 @@ func RetrieveServiceNetworkingNetworkName(d *schema.ResourceData, config *transp
 
 const parentServicePattern = "^services/.+$"
 
-// NOTE(craigatgoogle): An out of band aspect of this API is that it requires the service name to be
+// TODO: An out of band aspect of this API is that it requires the service name to be
 // formatted as "services/<serviceName>"
 func formatParentService(service string) string {
 	r := regexp.MustCompile(parentServicePattern)