var-naming rule can suggest invalid identifiers about revive

[chavacava]

Hi @jonathangjertsen , thanks for reporting the issue.

Using _ in identifiers is not idiomatic in Go, that is why the rule complains on record_

Sure, the proposed fix (record) is not valid because it results to be a reserved word of the language. It could be possible to avoid these kinds of incorrect recommendations by checking against the list of reserved words.
In these cases, it could be tricky to propose an alternative name. Thus, the rule could just signal the naming problem without proposing a fix.

IMO, adding these checks isn't worth the performance and complexity penalities.

In your case, you could just use something like strukt to avoid the names clash.

[chavacava]

I will close the issue for now, and if it turns out to be impacting a larger number of users, I'll reopen it.