status-im / nim-confutils

Simplified handling of command line options and config files
Apache License 2.0
63 stars 16 forks source link

fix BareExcept warning #67

Closed tersec closed 1 year ago

tersec commented 1 year ago

https://github.com/status-im/nimbus-eth2/issues/4661

nimbus-eth2/vendor/nim-confutils/confutils.nim(885, 5) Warning: The bare except clause is deprecated; use except CatchableError: instead [BareExcept]

https://github.com/status-im/nim-confutils/blob/38dfeaaabdc6792d0f4d701621cbe34001978456/confutils.nim#L903-L917

If nothing else, 1.6.x don't all have BareExcept -- the more correct comparisons are (NimMajor, NimMinor, NimPatch) >= (1, 6, 11) or >= (1, 6, 12) (hopefully soon).