Feature Request: Customize initialisms or remove "ID" from the common list about revive HOT 6 CLOSED

We can make the rule configurable. I'd prefer to keep this check under var-naming so we can be as compatible with golint as possible.

If you have some spare time, it'll be great if you can give it a try! We can discuss the approach you're planning here so we can collect some more thoughts! // cc @chavacava

from revive.

As described by @cvbarros, the configuration with an optional (empty by default) white list of "relaxed" initialisms seems the most flexible approach. It may lead to inconsistencies across projects but not more than adding a new rule (option 3).

from revive.

I'm thinking of two options:

• Allow developers to provide a custom list of initialisms. By default, we use the built-in one
• Allow developers to add extra valid Initialisms

The second aligns with what you mentioned, and the configuration will be less verbose in case the only initialisms a developer wants to alter is Id, so we can go with it.

from revive.

I think that 2 will also provide a less verbose, while still flexible approach. Feel free to work on it if you want, otherwise I'll give it a try next week 👍

from revive.

@cvbarros I won't be able to work on it shortly. Assigned the issue to you so you can take care.

from revive.

I implemented this feature right here.

Let me know if you have other suggestions.

from revive.