lint

.golangci.yml

@@ -25,3 +25,8 @@ linters:
     - unparam
     - unused
     - varcheck
+
+issues:
+  exclude:
+    # staticcheck
+    - "SA1019: request.GetSpec().GetInputs is deprecated: Do not use."

pkg/config/third_party_config.go

@@ -7,6 +7,6 @@ type ThirdPartyConfigOptions struct {
 }
 
 type FlyteClientConfig struct {
-	ClientID    string   `json:"clientId" pflag:",public identifier for the app which handles authorization for a Flyte deployment"`
-	RedirectURI string   `json:"redirectUri" pflag:",This is the callback uri registered with the app which handles authorization for a Flyte deployment"`
+	ClientID    string `json:"clientId" pflag:",public identifier for the app which handles authorization for a Flyte deployment"`
+	RedirectURI string `json:"redirectUri" pflag:",This is the callback uri registered with the app which handles authorization for a Flyte deployment"`
 }

pkg/manager/impl/execution_manager.go

@@ -735,6 +735,7 @@ func (m *ExecutionManager) CreateExecution(
 	*admin.ExecutionCreateResponse, error) {
 	// Prior to  flyteidl v0.15.0, Inputs was held in ExecutionSpec. Ensure older clients continue to work.
 	if request.Inputs == nil || len(request.Inputs.Literals) == 0 {
+		//lint:ignore SA1019
 		request.Inputs = request.GetSpec().GetInputs()
 	}
 	var executionModel *models.Execution