New Check: Preferred Naming of Parameters for CreateFunc, DeleteFunc, ReadFunc, and UpdateFunc

[bflad]

This check seems like it should be doable in AST using the function signature of func XXX(*schema.ResourceData, interface{}) error, however it may require SSA.

In resource CreateFunc, DeleteFunc, UpdateFunc, and UpdateFunc, it is preferable to name the first parameter of *schema.ResourceData to d and the second parameter of interface{} to meta.

Valid:

func XXX(d *schema.ResourceData, meta interface{}) error {
  // ...
}

Invalid:

func XXX(invalid *schema.ResourceData, meta interface{}) error {
  // ...
}

func XXX(d *schema.ResourceData, invalid interface{}) error {
  // ...
}