Skip to content

Conversation

@gqcn
Copy link
Member

@gqcn gqcn commented Jul 18, 2022

No description provided.

@gqcn gqcn requested review from hailaz, houseme and huangqian1985 July 18, 2022 07:54
@gqcn gqcn changed the title improve package gerror, add HasCode/HasError function for package gerror use method name as its command name if no name defined in Meta of input struct for package gcmd Jul 18, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jul 18, 2022

Codecov Report

Merging #2019 (4f06827) into master (5d51e9f) will increase coverage by 0.01%.
The diff coverage is 37.83%.

@@            Coverage Diff             @@
##           master    #2019      +/-   ##
==========================================
+ Coverage   76.49%   76.51%   +0.01%     
==========================================
  Files         515      515              
  Lines       46666    46658       -8     
==========================================
+ Hits        35698    35699       +1     
+ Misses       8998     8990       -8     
+ Partials     1970     1969       -1     
Flag Coverage Δ
go-1.15-386 76.42% <37.83%> (+<0.01%) ⬆️
go-1.15-amd64 76.42% <37.83%> (+0.01%) ⬆️
go-1.16-386 76.45% <37.83%> (+0.04%) ⬆️
go-1.16-amd64 76.46% <37.83%> (+0.04%) ⬆️
go-1.17-386 76.48% <37.83%> (+0.02%) ⬆️
go-1.17-amd64 76.47% <37.83%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
os/gcmd/gcmd_command_object.go 52.41% <34.28%> (+1.00%) ⬆️
os/gcron/gcron_entry.go 69.52% <100.00%> (ø)
os/gfpool/gfpool_file.go 65.71% <0.00%> (-2.86%) ⬇️
contrib/registry/polaris/polaris_watcher.go 73.21% <0.00%> (+5.35%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d51e9f...4f06827. Read the comment docs.

@gqcn gqcn merged commit b7794a8 into master Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants