search

Consensys / corset

4 stars 10 forks source link

Corset Warning for Empty Columns #90

Open DavePearce opened 1 month ago

DavePearce commented 1 month ago

Check whether Corset already provides a warning. If not, then add one. The reason being if some column is incorrectly named then we want to catch that.

delehef commented 1 month ago

Warning are already emitted; warnings can be turned into early exit by dynamically changing this false argument; e.g. by introducing a CLI switch.

DavePearce commented 1 month ago

Thanks! This is really just a reminder for me to look into it to see how the warnings are generated, and basically learn more about the tool :)