-
Notifications
You must be signed in to change notification settings - Fork 215
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
how to change lint rule "legacy-generate-region"? #2288
Comments
The rule specifically checks against
The rule could be generalized to take a regex (or a list of prefixes, ...). In case you're considering giving it a try, this might serve as an example: https://github.com/chipsalliance/verible/blob/master/verilog/analysis/checkers/enum_name_style_rule.cc |
This means that the user cannot modify the regular expression with test parameters, only by recompiling it to a new linter file, right? |
Right. Currently, there is no way of configuring the rule. But it can be made configurable. |
I think we should keep this open until implemented. |
Agree. Let's leave it open with a |
May I work on this issue |
Summary
//Brief description of request. Reference
[style-lint-rule-name]
if this is//about an existing rule, or suggest new lint rule name.
hello:
in rule file https://chipsalliance.github.io/verible/lint.html#generate-label-prefix , there is no parameter to user change the
style_regex
.May I ask whether modification is not supported or I have not found a way to modify
The text was updated successfully, but these errors were encountered: