-
Notifications
You must be signed in to change notification settings - Fork 212
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
Support for naming conventions #310
Comments
I'm hearing similar requests coming from some teams. I acknowledge not everyone will share the same convention, but fortunately, we've introduced rule configurability so it should be possible to pass regular expressions to the rules that check conventions. @msfschaffer I believe you wanted this too, right? To point to one example: The currently hardcoded style checks could come from rule configurations. Here's an example of a configurable rule: |
Hi, so if I understand it correctly we would like to have the whole name to be configured with regex or just only some parts of the gathered name should be verified with regex? What regex implementation should I use, the one from the cpp standard? |
I think the whole name would allow for most flexibility. In the example I gave we want to check that input ports of the module always start with The configuration for that could then be something like: I'm using PCRE style regexes here but, for me at least, as long as it supports the basics and is well documented, the exact syntax is not that important. I guess that is up to @fangism (or other maintainers) to decide. |
Ok, thanks for the clarification. |
Are there any plans to add linting rules for naming conventions?
For example module inputs should start with
i_
or should end with_i
.Preferably this would be configurable using some kind of template.
The text was updated successfully, but these errors were encountered: