-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Outstanding cleanup items #1054
Comments
Merged
After the Nim 2.0.14 there are more: Warning: imported and not used
Warning: `stew/results` is now available as `import results` via the `results` Nimble package; results is deprecated [Deprecated]
Warning: template 'activeChroniclesScope' is implicitly redefined; this is deprecated, add an explicit .redefine pragma [ImplicitTemplateRedefinition]
Warning: use std/net; net is deprecated [Deprecated]
Warning: use exitprocs.addExitProc; addQuitProc is deprecated [Deprecated]
Warning: use {.base.} for base methods; baseless methods are deprecated [UseBase]
Warning: catch a more precise Exception deriving from CatchableError or Defect. [BareExcept]
Warning: Callback must not raise exceptions, annotate with {.async: (raises: []).}; createStreamServer is deprecated [Deprecated]
Warning: unreachable code after 'return' statement or '{.noReturn.}' proc [UnreachableCode]
Other
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In my cleanup work I mainly focused on removing unused imports, but there are 3 other types of issues that should get cleanuped:
Warning: use treeCid instead; cid is deprecated [Deprecated]
General
Warning: The raises pragma doesn't work on async procedures - use `async: (raises: [...]) instead. [User]
The text was updated successfully, but these errors were encountered: