status-im / nimbus-eth2

Nim implementation of the Ethereum Beacon Chain
https://nimbus.guide
Other
543 stars 233 forks source link

replace default word usage in REST serializaton #6580

Closed pedromiguelmiranda closed 1 month ago

pedromiguelmiranda commented 1 month ago

Given that default is already a template defined by the Nim standard library, it's less confusing to use a different name.

Removed the extra "var default:" declaration.

github-actions[bot] commented 1 month ago

Unit Test Results

         9 files  ±0    1 355 suites  ±0   39m 26s :stopwatch: -28s   5 150 tests ±0    4 802 :heavy_check_mark: ±0  348 :zzz: ±0  0 :x: ±0  21 525 runs  ±0  21 121 :heavy_check_mark: ±0  404 :zzz: ±0  0 :x: ±0 

Results for commit 6b8b8c10. ± Comparison against base commit 31b5c3e3.