rack icon indicating copy to clipboard operation
rack copied to clipboard

chore: unnecessary use of fmt.Sprintf

Open testwill opened this issue 2 years ago • 1 comments

What is the feature/fix?

** Describe the task and what is the goal for it. ** ** Describe the bug and the solution. ** ** Any additional information **

Add screenshot or video (optional)

** Any screenshot or video capture using the feature **

Does it has a breaking change?

** Describe the changes and if it has any breaking changes in any feature **

How to use/test it?

** Describe how to test or use the feature **

Checklist

  • [ ] New coverage tests
  • [ ] Unit tests passing
  • [ ] E2E tests passing
  • [ ] E2E downgrade/update test passing
  • [ ] Documentation updated
  • [ ] No warnings or errors on Deepsource/Codecov

testwill avatar Aug 31 '23 13:08 testwill

Codecov Report

Attention: Patch coverage is 82.14286% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 33.58%. Comparing base (8e3368f) to head (8ec9e78).

Files Patch % Lines
pkg/manifest1/process.go 0.00% 1 Missing :warning:
pkg/router/controller_deployment.go 0.00% 1 Missing :warning:
pkg/router/controller_ingress.go 0.00% 1 Missing :warning:
pkg/router/router.go 0.00% 1 Missing :warning:
sdk/methods.go 95.00% 1 Missing :warning:
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3698   +/-   ##
=======================================
  Coverage   33.58%   33.58%           
=======================================
  Files         180      180           
  Lines       20220    20220           
=======================================
  Hits         6790     6790           
  Misses      12284    12284           
  Partials     1146     1146           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Aug 31 '23 13:08 codecov[bot]