PascalCase/camelCase and acronyms about vhdl-style-guide HOT 5 OPEN

Evening @maltaisn ,

We should be able to allow the user to define a regex. Are you thinking we would add another option like regex_exceptions which would be a list of regexs?

I am curious what your regexp would be to cover PipelineSR, SPIAccess, and ADCRegisters.

Regards,

--Jeremy

from vhdl-style-guide.

In my case the regex wouldn't check for known acronyms, I would go for something simple such as [A-Z][A-Za-z\d]*.

from vhdl-style-guide.

Morning @maltaisn ,

I am trying to figure out how the configuration would work to support a user regex. I think I will add add another option to case and then add an option for regex.

So in your regex example you would use the following configuration:

{
  "rule": {
    "signal_002": {
      "case": "regex",
      "regex": [
         "[A-Z][A-Za-z\d]*"
      ]
    }
  }
}

The prefix, suffix, and case exceptions would still apply.

Would that work for you?

--Jeremy

from vhdl-style-guide.

That would be perfect, thanks!

from vhdl-style-guide.

Evening @maltaisn ,

I pushed an update to the issue-1182 branch. I am only allowing a single regex instead of a list of regexs. When you get a chance could you check it out and let me know if it works for you?

Thanks,

--Jeremy

from vhdl-style-guide.