Require statements don't have reason

[maxsam4]

We should add reason along with all the require statements so that curious users can debug on their own.

This was not done earlier as it would've increased the deployment cost significantly but as we are using proxies now, It doesn't matter. A contract is deployed only once.

[maxsam4]

Note: We don't need to add whole text reason (Strings take a lot of gas and block gas limit 👎 )
We should add an error code and maintain a wiki of error codes -> human readable reasons.

[kashishkhullar]

Could you comment the link of the wiki

[reemaSomish]

Hi @maxsam4 ! Thank you for informing us, we are checking the issue.