CR changes

Author: EyalDelarea

common/commands/command.go

@@ -39,10 +39,10 @@ func Exec(command Command) error {
 	return err
 }
 
-// ExecAndReportUsage runs the command and then triggers a usage report if needed,
+// ExecAndThenReportUsage runs the command and then triggers a usage report
 // Is used for commands which don't have the full server details before execution
 // For example: oidc exchange command, which will get access token only after execution.
-func ExecAndReportUsage(cc Command) (err error) {
+func ExecAndThenReportUsage(cc Command) (err error) {
 	if err = cc.Run(); err != nil {
 		return
 	}

common/commands/config.go

@@ -234,7 +234,7 @@ func exchangeOidcTokenAndSetAccessToken(cc *ConfigCommand) error {
 		SetRunId(cc.oidcSetupParams.RunId)
 
 	// Usage report will be sent only after execution in order to have valid token
-	err := ExecAndReportUsage(exchangeOidcTokenCmd)
+	err := ExecAndThenReportUsage(exchangeOidcTokenCmd)
 	if err != nil {
 		return err
 	}

general/token/oidctokenexchange.go

@@ -57,7 +57,7 @@ type OidcTokenParams struct {
 	ProviderName string
 	TokenId      string
 	Audience     string
-	// Those values are used to link the token to a specific use, they are optional
+	// These values are used to link the token to a specific use, they are optional
 	ProjectKey     string
 	ApplicationKey string
 	JobId          string