search

nspcc-dev / neo-go

Go Node and SDK for the Neo blockchain
MIT License
123 stars 79 forks source link

Linter: adapt org-wide linter #3592

Closed AliceInHunterland closed 1 month ago

AliceInHunterland commented 1 month ago

Close #3591, #3417.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 44.44444% with 15 lines in your changes missing coverage. Please review.

Project coverage is 85.27%. Comparing base (e1d5ac8) to head (cf2e629). Report is 6 commits behind head on master.

Files with missing lines Patch % Lines
pkg/smartcontract/rpcbinding/binding.go 20.00% 4 Missing :warning:
pkg/compiler/codegen.go 50.00% 3 Missing :warning:
pkg/services/rpcsrv/server.go 0.00% 3 Missing :warning:
pkg/neorpc/rpcevent/filter.go 0.00% 2 Missing :warning:
pkg/consensus/recovery_message.go 0.00% 1 Missing :warning:
pkg/network/server.go 0.00% 1 Missing :warning:
pkg/vm/vm.go 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #3592 +/- ## ========================================== + Coverage 85.22% 85.27% +0.04% ========================================== Files 333 333 Lines 38984 39005 +21 ========================================== + Hits 33224 33260 +36 + Misses 4191 4173 -18 - Partials 1569 1572 +3 ```

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

roman-khimov commented 1 month ago

Gah, too many commits now. One for exhaustive, one for fmt, like that.