search

yarpc / yarpc-go

A message passing platform for Go
MIT License
401 stars 101 forks source link

Apply all linter fixes and regenerate code #2260

Closed biosvs closed 2 months ago

biosvs commented 2 months ago

Linter build was broken and ignored for a long time, so a lot of code should be upgraded to comply with linter and generators output.

Most of the changes are just formatting, comments, etc.

The only change that can be observed is Fault type from yarpcerrors/yarpcerrorclassifier.go which become pubic (because it's returned from public methods).

Note: this PR doesn't address performance, improvements, etc. The only task is to get green build with least changes.

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.22%. Comparing base (f2744e8) to head (ba74cd3).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #2260 +/- ## ========================================== - Coverage 85.93% 85.22% -0.72% ========================================== Files 266 270 +4 Lines 15423 15555 +132 ========================================== + Hits 13254 13256 +2 - Misses 1746 1876 +130 Partials 423 423 ```

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