-
Notifications
You must be signed in to change notification settings - Fork 357
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
Removing well-proven and therefore outdated asserts #2616
Conversation
…oven to never cause an issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just minor suggestions regarding the removal (and one addition!) of empty lines resulting from the changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Beautiful, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for cleaning this up!
Good point! I just checked nest-extension-module repository and we seem to have dropped them already there. |
The NESTML templates also do not use these assert statements. |
The asserts in the
Node::update
method of all models derived fromNode
were checking if the parameters for the requested update interval are indeed correct. These asserts are now generally seen as superfluous, as they have not been triggered in many years now.