Consensys / vscode-solidity-auditor

Solidity language support and visual security auditor for Visual Studio Code
https://marketplace.visualstudio.com/items?itemName=tintinweb.solidity-visual-auditor
GNU General Public License v3.0
569 stars 60 forks source link

Display compilation errors in my Solidity code #81

Closed lcswillems closed 2 years ago

lcswillems commented 2 years ago

Hi!

Something I like about this VSCode Solidity extension is that it shows me compilation errors directly in my Solidity code.

I don't think it is possible in your VSCode extension. It would be nice to have such a feature.

tintinweb commented 2 years ago

Hi @lcswillems 🙌

would be nice but the extension is only parsing your code without actually compiling it. nice side-effect: it's way faster than extensions that actually compile code + we can allow for an error of margin without having to bug you while typing.

the extension, however, is compatible with the vscode-solidity extension. should be fine to install both :)

cheers, tin

lcswillems commented 2 years ago

"nice side-effect: it's way faster than extensions that actually compile code + we can allow for an error of margin without having to bug you while typing." : I don't think it is a nice side-effect, I think it is more like a big limitation from my point of view. It is not way faster than VSCode Solidity extension for example.

Why this extension is only "only parsing your code without actually compiling it"? Why this limitation? You caracterize your extension as "An extension that supports developers in writing secure and well understood code". Displaying errors as soon as they are written seems to go in this direction.

tintinweb commented 2 years ago

since this extension is not compiling your code it, unfortunately, cannot provide you with compiler output. however, as noted, the two extensions are compatible which should allow you to get the best of both worlds 🙌

cheers, tin

lcswillems commented 2 years ago

Okay, thank you for the explanations! I am going to use both as you suggest