Skip to content
This repository has been archived by the owner on Dec 31, 2024. It is now read-only.

We need error messages for revert(), require(), and assert() #12

Open
GriffGreen opened this issue Sep 29, 2017 · 1 comment
Open

We need error messages for revert(), require(), and assert() #12

GriffGreen opened this issue Sep 29, 2017 · 1 comment

Comments

@GriffGreen
Copy link
Member

The entire contract needs to utilize solidity's new functionality. This might be a fun way to let community members contribute to the project.

Each error message should be it's own PR.

See https://media.consensys.net/when-to-use-revert-assert-and-require-in-solidity-61fb2c0e5a57 for more background info

@djudjuu
Copy link

djudjuu commented Jul 8, 2019

hey @GriffGreen ,
let's specidy this a bit more: What style do you want those error-messages to be

example: require(donorAddress != 0); could be

a) donor address must not be zero
b) donor address should be nonzero
c) donor address is zero address

plus variations on capitalization and punctuation (dots at the end or not?)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants