verible-verilog-lint not works - Githubissues

TerosTechnology / vscode-terosHDL

VHDL and Verilog/SV IDE: state machine viewer, linter, documentation, snippets... and more!

https://terostechnology.github.io/terosHDLdoc/

GNU General Public License v3.0

570 stars 46 forks source link

verible-verilog-lint not works #257

Closed tarik-jin closed 3 years ago

tarik-jin commented 3 years ago

Describe the bug refer to https://terostechnology.github.io/terosHDLdoc/style/configuration.html it seems nothing was reported after config while i could manual execute successfully and report sytle errors locally

Please complete the following information:

OS: [e.g.Ubuntu 20.04]
VSCode version version Version: 1.61.0 Commit: ee8c7def80afc00dd6e593ef12f37756d8f504ea Date: 2021-10-07T18:11:02.929Z Electron: 13.5.1 Chrome: 91.0.4472.164 Node.js: 14.16.0 V8: 9.1.269.39-electron.0 OS: Linux x64 5.11.0-37-generic

Additional context wonder if I could see some messages output when i specify verible-verilog-lint in style checker configuration page

qarlosalberto commented 3 years ago

Double check: have you configured the Verible binary folder path? What is?

tarik-jin commented 3 years ago

don' know why linter/style checker not work...

tarik-jin commented 3 years ago

linters settings page

qarlosalberto commented 3 years ago

Got it! There is a bug when the "--rules" arguments is empty. I will fixed in the next version. Meanwhile you can add a rule to the argument.

qarlosalberto commented 3 years ago

Try with this beta version: https://github.com/TerosTechnology/vscode-terosHDL/releases/tag/latest

tarik-jin commented 3 years ago

seems not working, or there is something wrong in my configuration?

qarlosalberto commented 3 years ago

Go to: Help -> Toggle Developer Tools -> Console. You should see a message similar to:

Copy a capture here. Also, could you share the code? So I can reproduce it.

tarik-jin commented 3 years ago

seems verible only reports when there is syntax error

with out syntax error, trailing-spaces warning was not reported

qarlosalberto commented 3 years ago

Now it should be fixed. Try the beta version.

tarik-jin commented 3 years ago

try the latest beat version, seems some errors in console output?

qarlosalberto commented 3 years ago

Could you try again? Please, tell me the name of the vsix file.

tarik-jin commented 3 years ago

uninstall the previous vsix, use this version

vscode version Version: 1.61.1 Commit: c13f1abb110fc756f9b3a6f16670df9cd9d4cf63 Date: 2021-10-14T01:15:35.620Z Electron: 13.5.1 Chrome: 91.0.4472.164 Node.js: 14.16.0 V8: 9.1.269.39-electron.0 OS: Linux x64 5.11.0-37-generic

demo used (with a trailing space shown in following capture) module veribleTest ( input a, output b );

assign b = a;

endmodule

qarlosalberto commented 3 years ago

I'm trying to reproduce it. When have yo that errors? When you open a file? When you set the settings?...

qarlosalberto commented 3 years ago

Try again: 2021-10-16__03.08_teroshdl-2.0.3.vsix

tarik-jin commented 3 years ago

Try again: 2021-10-16__03.08_teroshdl-2.0.3.vsix

it works, thanks