I just upgraded to Nim 1.6.12 after a time away from working with Nim and one 
of my projects now gives lots of warnings when I run the project's tests (which 
all pass). The warnings are all of the form

> ~/.choosenim/toolchains/nim-1.6.12/lib/pure/unittest.nim(552, 5) Warning: The 
> bare except clause is deprecated; use `except CatchableError:` instead 
> [BareExcept]

There is of course a bare except in `unittest.nim` at that location, but as 
it's part of the standard library and there are no bare except clauses in my 
code I'm not sure what the best resolution is. I see the recommendation in the 
`unittest` docs to use `Testament` instead, but that would require changing all 
my working test code, right? I'd like to avoid that. Would it be acceptable to 
submit a PR for `unittest` changing the bare except to `except CatchableError:`?

Reply via email to