hardhat icon indicating copy to clipboard operation
hardhat copied to clipboard

Handle `info` messages in solidity

Open fvictorio opened this issue 4 years ago • 3 comments

At the moment, solidity has two kind of "messages" after a compilation: warnings and errors. But a third one, info, will be added. See this issue.

It seems like this won't break hardhat. The only problem is that these info messages are shown in yellow (as warnings). At the very least we should change that, but we might also have to consider things like allowing users to enable/disable different levels of verbosity, or configure them in some other way.

fvictorio avatar Sep 01 '21 13:09 fvictorio

This issue was marked as stale because it didn't have any activity in the last 30 days. If you think it's still relevant, please leave a comment indicating so. Otherwise, it will be closed in 7 days.

github-actions[bot] avatar Jul 29 '22 23:07 github-actions[bot]

I think it's still relevant. The compiler does emit these and they should be handled (though currently there are not many cases in which they appear, mostly in SMTChecker I think).

cameel avatar Aug 01 '22 11:08 cameel

It is indeed, thank you.

fvictorio avatar Aug 08 '22 09:08 fvictorio

@cameel do you have an example of some code that produces an info message in some solc version?

fvictorio avatar Dec 29 '22 07:12 fvictorio

I think that currently the only one is error 1180 from SMT checker. Check out the command-line test standard_model_checker_invariants_contract.

We do plan to use it more and maybe convert some of the existing warnings into info messages but this has been a low priority so far. May take some time until we focus specifically on that unless this comes up in one of the features we're currently adding or someone explicitly requests some warning to be demoted.

cameel avatar Dec 29 '22 12:12 cameel

Thanks for the clarification. I'm going to mark this as blocked until there is some info message in solc that can show up in a normal Hardhat usage.

fvictorio avatar Dec 29 '22 13:12 fvictorio