fixed typo in comments (#2997)

Signed-off-by: Youngjun <yj.yoo@okestro.com>

pkg/porter/build.go

@@ -62,7 +62,7 @@ func (o *BuildOptions) Validate(p *Porter) error {
 		return fmt.Errorf("invalid --driver value %s", o.Driver)
 	}
 
-	// Syncing value back to the config and we will always use the config
+	// Syncing value back to the config, and we will always use the config
 	// to determine the driver
 	// This would be less awkward if we didn't do an automatic build during publish
 	p.Data.BuildDriver = o.Driver

pkg/porter/lifecycle.go

@@ -158,7 +158,7 @@ func (o *BundleExecutionOptions) defaultDriver(p *Porter) {
 	//
 	// When you run porter installation apply, there are some settings from porter install
 	// that aren't exposed as flags (like driver and allow-docker-host-access).
-	// This allows the user to set them in the config file and we will use them before running the bundle.
+	// This allows the user to set them in the config file, and we will use them before running the bundle.
 	//
 
 	// Apply global config to the --driver flag

pkg/porter/list.go

@@ -179,7 +179,7 @@ func (d DisplayInstallation) ConvertToInstallation() (storage.Installation, erro
 	return i, nil
 }
 
-// ConvertParamToSet converts a Parameters into a internal ParameterSet.
+// ConvertParamToSet converts a Parameters into an internal ParameterSet.
 func (d DisplayInstallation) ConvertParamToSet() (storage.ParameterSet, error) {
 	strategies := make([]secrets.SourceMap, 0, len(d.Parameters))
 	for name, value := range d.Parameters {

pkg/porter/publish.go

@@ -84,7 +84,7 @@ func (o *PublishOptions) validateTag() error {
 }
 
 // Publish is a composite function that publishes an invocation image, rewrites the porter manifest
-// and then regenerates the bundle.json. Finally it publishes the manifest to an OCI registry.
+// and then regenerates the bundle.json. Finally, it publishes the manifest to an OCI registry.
 func (p *Porter) Publish(ctx context.Context, opts PublishOptions) error {
 	ctx, log := tracing.StartSpan(ctx)
 	defer log.EndSpan()

pkg/porter/reconcile.go

@@ -145,7 +145,7 @@ func (p *Porter) IsInstallationInSync(ctx context.Context, i storage.Installatio
 		}
 	}
 
-	// We want to upgrade but we don't have values to compare against
+	// We want to upgrade, but we don't have values to compare against
 	// This shouldn't happen but check just in case
 	if lastRun == nil {
 		log.Info("Triggering because the last run for the installation wasn't recorded")