search

JuliaIO / ProtoBuf.jl

Julia protobuf implementation
Other
205 stars 55 forks source link

update export format #207

Closed zsz00 closed 2 years ago

zsz00 commented 2 years ago

like this: image

codecov[bot] commented 2 years ago

Codecov Report

Merging #207 (7526656) into master (491d4c8) will decrease coverage by 0.02%. The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
- Coverage   88.82%   88.79%   -0.03%     
==========================================
  Files          25       25              
  Lines        2782     2785       +3     
==========================================
+ Hits         2471     2473       +2     
- Misses        311      312       +1
Impacted Files Coverage Δ
src/codegen/toplevel_definitions.jl 95.77% <87.50%> (-0.63%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Drvi commented 2 years ago

Thanks for the contribution! I tweaked the code slightly so that there is less indexing and fewer string concatenations. The formatted exports are a nice improvement!