You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
localparam_style Default: CamelCase Style of localparam names parameter_style Default: CamelCase|ALL_CAPS Style of parameter names.
But that confuses camelCase with PascalCase. Just from the documentation, it is not clear which of these code examples would be the lint default (lowercase "b" versus uppercase "B")
The same page includes a link to https://github.com/chipsalliance/verible/tree/master/verilog/tools/lint#regular-expressions referencing other common naming styles. It is correctly shown as camelCase not not CamelCase on that page. Another inconsistency is it's called ALL_CAPS in the former but UPPER_SNAKE_CASE in the latter. Full list from the regex reference page:
For naming style rules, the regex can be kept simple and not have to worry about the beginning starting with a digit as this is already taken care of by the lexer/parser. Below are some common naming style regex patterns that can be used.
Case
Regex
lower_snake_case
[a-z_0-9]+
UPPER_SNAKE_CASE
[A-Z_0-9]+
Title_Snake_Case
[A-Z]+[a-z0-9](_[A-Z0-9]+[a-z0-9])*
Sentence_snake_case
([A-Z0-9]+[a-z0-9]_?)([a-z0-9]_)
camelCase
([a-z0-9]+[A-Z0-9]*)+
PascalCase
([A-Z0-9]+[a-z0-9]*)+
The text was updated successfully, but these errors were encountered:
https://chipsalliance.github.io/verible/verilog_lint.html includes some information on style defaults:
But that confuses
camelCase
withPascalCase
. Just from the documentation, it is not clear which of these code examples would be the lint default (lowercase "b" versus uppercase "B")The same page includes a link to https://github.com/chipsalliance/verible/tree/master/verilog/tools/lint#regular-expressions referencing other common naming styles. It is correctly shown as
camelCase
not notCamelCase
on that page. Another inconsistency is it's calledALL_CAPS
in the former butUPPER_SNAKE_CASE
in the latter. Full list from the regex reference page:The text was updated successfully, but these errors were encountered: