okteto: Need better error handling

Describe the bug Need error handling here: https://github.com/okteto/okteto/blob/39b5b7757668a2b4bea7b3e0054c3b174f010dd9/internal/test/utils.go#L36

Additional context Need to check if SetEnv() func returns an error. Possibly correct it through updating linters. Context: https://kubernetes.slack.com/archives/CM1QMQGS0/p1667276346405729

About this issue

  • Original URL
  • State: closed
  • Created 2 years ago
  • Comments: 17 (16 by maintainers)

Most upvoted comments

I think oktetoLog.Infof() or oktetoLog.Debugf() are the best functions to use since it’s not an error that we should show to the user

Awesome @abs007! We are using deepsource here but happy to update its config or even use a different one. Sync with @jLopezbarb for proposals