lint: Configure one of our linters to check for a 100 character line length limit

lowRISC / ibex

Ibex is a small 32 bit RISC-V CPU core, previously known as zero-riscy.

https://www.lowrisc.org

Apache License 2.0

1.38k stars 543 forks source link

lint: Configure one of our linters to check for a 100 character line length limit #1027

Closed imphil closed 2 years ago

imphil commented 4 years ago

The title says it all ... AscentLint complains about it, but seemingly no of our other linters in Ibex cares. I'd expect there to be a verible lint option we just need to turn on.

imphil commented 4 years ago

See the OT configuration at https://github.com/lowRISC/opentitan/blob/master/hw/lint/tools/veriblelint/rules.vbl. We'll probably get this config for free once we vendor the latest changes in from OT.

GregAC commented 2 years ago

Verible lint is now running in CI