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.

Sign up for GitHub

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

Jump to bottom

Add message argument to error handling functions #398

Closed
jijordre opened this issue May 21, 2019 · 1 comment
Closed

Add message argument to error handling functions #398

jijordre opened this issue May 21, 2019 · 1 comment
Assignees
@jijordre
Labels
enhancement

Comments

@jijordre
Copy link
Contributor

Description

Error handling functions like require() and revert() are currently not called with the optional error/explanation message string. The task of this issue is to add relevant message arguments to error handling functions calls where applicable.

Definition of done

  • Error handling function calls have been updated
  • Unit tests run

Details & Questions

Depends on
@jijordre jijordre self-assigned this May 21, 2019
@jijordre jijordre changed the title Add message argument to require() calls Add message argument to error handling functions May 21, 2019
@jijordre jijordre mentioned this issue May 21, 2019
Merged
@jijordre jijordre mentioned this issue Jun 8, 2019
Merged
@jijordre
Copy link
Contributor Author

jijordre commented Jun 8, 2019

Closed by #400.

@jijordre jijordre closed this as completed Jun 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jijordre jijordre

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jijordre