Skip to content
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

Incorrectly use raises annotations #7

Open
beef331 opened this issue Sep 26, 2023 · 2 comments
Open

Incorrectly use raises annotations #7

beef331 opened this issue Sep 26, 2023 · 2 comments

Comments

@beef331
Copy link
Contributor

beef331 commented Sep 26, 2023

Presently procedures are annotated raises: [SomeDefect] this causes warnings cause raise is only meant for CatchableErrors there is no Defect tracking.

@termermc
Copy link
Owner

I'm surprised that there's no Defect tracking. I'll figure this out a bit later, wish I saw this before making the last release.

@beef331
Copy link
Contributor Author

beef331 commented Sep 28, 2023

You could tag your procs with your own Defective then on any consuming modules push forbids: [Defective], but yea that aside there is nothing.

https://nim-lang.org/docs/manual.html#effect-system-tag-tracking

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

No branches or pull requests

2 participants