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

Add new AssertionError Exception type #9734

Merged
merged 1 commit into from
Feb 13, 2015
Merged

Add new AssertionError Exception type #9734

merged 1 commit into from
Feb 13, 2015

Conversation

jakebolewski
Copy link
Member

  • assert() and @assert throw a more specific AssertionError type
  • add tests, update docs, add NEWS.md entry

* add `assert` and `@assert` tests
* make `assert` take an optional message argument
* update doc and NEWS.md
@jakebolewski
Copy link
Member Author

This is a simpler version. I added tests and updated the docs. Currently the docs say that assert method can take an optional message argument which is currently not true.


Raise an error if ``cond`` is false. Preferred syntax for writings assertions.
Throw an ``AssertionError`` if ``cond`` is false. Preferred syntax for writings assertions.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was there before, but should be "writing assertions" not "writings assertions"

jakebolewski added a commit that referenced this pull request Feb 13, 2015
Add new `AssertionError` Exception type
@jakebolewski jakebolewski merged commit d78eff1 into master Feb 13, 2015
@jakebolewski jakebolewski deleted the jcb/asserterr branch February 13, 2015 14:00
StefanKarpinski pushed a commit that referenced this pull request Feb 8, 2018
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.

None yet

2 participants