Merge branch '5.1' into 5.2

* 5.1:
  Update .php_cs.dist
  Apply "visibility_required" CS rule to constants

Author: nicolas-grekas

Argument/BoundArgument.php

@@ -16,9 +16,9 @@
  */
 final class BoundArgument implements ArgumentInterface
 {
-    const SERVICE_BINDING = 0;
-    const DEFAULTS_BINDING = 1;
-    const INSTANCEOF_BINDING = 2;
+    public const SERVICE_BINDING = 0;
+    public const DEFAULTS_BINDING = 1;
+    public const INSTANCEOF_BINDING = 2;
 
     private static $sequence = 0;
 

Compiler/PassConfig.php

@@ -22,11 +22,11 @@
  */
 class PassConfig
 {
-    const TYPE_AFTER_REMOVING = 'afterRemoving';
-    const TYPE_BEFORE_OPTIMIZATION = 'beforeOptimization';
-    const TYPE_BEFORE_REMOVING = 'beforeRemoving';
-    const TYPE_OPTIMIZE = 'optimization';
-    const TYPE_REMOVE = 'removing';
+    public const TYPE_AFTER_REMOVING = 'afterRemoving';
+    public const TYPE_BEFORE_OPTIMIZATION = 'beforeOptimization';
+    public const TYPE_BEFORE_REMOVING = 'beforeRemoving';
+    public const TYPE_OPTIMIZE = 'optimization';
+    public const TYPE_REMOVE = 'removing';
 
     private $mergePass;
     private $afterRemovingPasses = [];

ContainerInterface.php

@@ -24,11 +24,11 @@
  */
 interface ContainerInterface extends PsrContainerInterface
 {
-    const RUNTIME_EXCEPTION_ON_INVALID_REFERENCE = 0;
-    const EXCEPTION_ON_INVALID_REFERENCE = 1;
-    const NULL_ON_INVALID_REFERENCE = 2;
-    const IGNORE_ON_INVALID_REFERENCE = 3;
-    const IGNORE_ON_UNINITIALIZED_REFERENCE = 4;
+    public const RUNTIME_EXCEPTION_ON_INVALID_REFERENCE = 0;
+    public const EXCEPTION_ON_INVALID_REFERENCE = 1;
+    public const NULL_ON_INVALID_REFERENCE = 2;
+    public const IGNORE_ON_INVALID_REFERENCE = 3;
+    public const IGNORE_ON_UNINITIALIZED_REFERENCE = 4;
 
     /**
      * Sets a service.

Dumper/PhpDumper.php

@@ -56,12 +56,12 @@ class PhpDumper extends Dumper
     /**
      * Characters that might appear in the generated variable name as first character.
      */
-    const FIRST_CHARS = 'abcdefghijklmnopqrstuvwxyz';
+    public const FIRST_CHARS = 'abcdefghijklmnopqrstuvwxyz';
 
     /**
      * Characters that might appear in the generated variable name as any but the first character.
      */
-    const NON_FIRST_CHARS = 'abcdefghijklmnopqrstuvwxyz0123456789_';
+    public const NON_FIRST_CHARS = 'abcdefghijklmnopqrstuvwxyz0123456789_';
 
     private $definitionVariables;
     private $referenceVariables;

Loader/Configurator/AbstractConfigurator.php

@@ -21,7 +21,7 @@
 
 abstract class AbstractConfigurator
 {
-    const FACTORY = 'unknown';
+    public const FACTORY = 'unknown';
 
     /**
      * @var callable(mixed, bool $allowService)|null

Loader/Configurator/AliasConfigurator.php

@@ -18,7 +18,7 @@
  */
 class AliasConfigurator extends AbstractServiceConfigurator
 {
-    const FACTORY = 'alias';
+    public const FACTORY = 'alias';
 
     use Traits\DeprecateTrait;
     use Traits\PublicTrait;

Loader/Configurator/ContainerConfigurator.php

@@ -27,7 +27,7 @@
  */
 class ContainerConfigurator extends AbstractConfigurator
 {
-    const FACTORY = 'container';
+    public const FACTORY = 'container';
 
     private $container;
     private $loader;

Loader/Configurator/DefaultsConfigurator.php

@@ -19,7 +19,7 @@
  */
 class DefaultsConfigurator extends AbstractServiceConfigurator
 {
-    const FACTORY = 'defaults';
+    public const FACTORY = 'defaults';
 
     use Traits\AutoconfigureTrait;
     use Traits\AutowireTrait;

Loader/Configurator/InlineServiceConfigurator.php

@@ -18,7 +18,7 @@
  */
 class InlineServiceConfigurator extends AbstractConfigurator
 {
-    const FACTORY = 'service';
+    public const FACTORY = 'service';
 
     use Traits\ArgumentTrait;
     use Traits\AutowireTrait;

Loader/Configurator/InstanceofConfigurator.php

@@ -18,7 +18,7 @@
  */
 class InstanceofConfigurator extends AbstractServiceConfigurator
 {
-    const FACTORY = 'instanceof';
+    public const FACTORY = 'instanceof';
 
     use Traits\AutowireTrait;
     use Traits\BindTrait;