Skip to content

Annotate all raises - #455

Merged
nitely merged 3 commits into
nim-lang:masterfrom
nitely:strict_raises
Sep 16, 2026
Merged

nitely merged 3 commits into
nim-lang:masterfrom
nitely:strict_raises

Conversation

@nitely

@nitely nitely commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Changes:

  • annotate all raises
  • add missing exception handlers
  • replace except Exception -> except CatchableError except for the main one
  • remove {.cast(raises: []).} and utils addCallback
  • get rid of some KeyError + add missing regression tests

@nitely nitely changed the title Strict raises Annotate all raises Sep 14, 2026
@nitely nitely mentioned this pull request Sep 14, 2026
@nitely
nitely force-pushed the strict_raises branch 2 times, most recently from 816575b to f56aaad Compare September 15, 2026 22:51
@nitely
nitely marked this pull request as ready for review September 16, 2026 22:15
@nitely
nitely merged commit 745e093 into nim-lang:master Sep 16, 2026
4 checks passed
@nitely
nitely deleted the strict_raises branch September 16, 2026 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant