Move specific rules from bleedingEdge to on by default

[mglaman]

Feature request

Proposals

		rules:
			testClassSuffixNameRule: true
			dependencySerializationTraitPropertyRule: true
			accessResultConditionRule: true
			cacheableDependencyRule: true
			hookRules: true
			loggerFromFactoryPropertyAssignmentRule: true
			entityStorageDirectInjectionRule: true
			symfonyYamlParseRule: true
			entityOperationsCacheabilityRule: true

hookRules may need to be renamed as it's pretty generic, my fault for not catching that.

[dpi]

Thought I'd check this out earlier.

For those looking into this, can be enabled under parameters.drupal, in case its not obvious. As in:

parameters:
  drupal:
    rules:
      testClassSuffixNameRule: true
      dependencySerializationTraitPropertyRule: true
      accessResultConditionRule: true
      cacheableDependencyRule: true
      hookRules: true
      loggerFromFactoryPropertyAssignmentRule: true
      entityStorageDirectInjectionRule: true
      symfonyYamlParseRule: true
      entityOperationsCacheabilityRule: true

And you will see errors for this full list for versions lower than 2.0.14,. so update!:

Note: Using configuration file [snip] phpstan.neon.
Invalid configuration:
Unexpected item 'parameters › drupal › rules › loggerFromFactoryPropertyAssignmentRule'.
Invalid configuration:
Unexpected item 'parameters › drupal › rules › entityStorageDirectInjectionRule'.
Invalid configuration:
Unexpected item 'parameters › drupal › rules › symfonyYamlParseRule'.
Invalid configuration:
Unexpected item 'parameters › drupal › rules › entityOperationsCacheabilityRule'.

[dpi]

Ran the new rules with two decently sized projects and other than #983, things seem alright.