-
Notifications
You must be signed in to change notification settings - Fork 35
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
Always panic through utils.Assert #112
Comments
this would be nice to expose as a flag like |
@guergabo I'm interested to work on this enhancement, can i work on this issue |
@KannappanChida. of course. lmk if you need any help. thank you ! |
interesting approach at sqlite. They have different behaviors for asserts depending on the build: https://www.sqlite.org/assert.html#:~:text=In%20SQLite%2C%20the%20presence%20of,about%20the%20truth%20of%20X. |
@KannappanChida - curious if you are still interested in working on this? |
Sometimes when we encounter an intolerable error we call
panic()
directly, instead we could callutils.Assert(false, ...)
that way we would have the opportunity to optionally turn off strict panicing in production.The text was updated successfully, but these errors were encountered: