Merge branch '5.1' into 5.2

* 5.1:
  Cleanup CI scripts
  fix code style
  fix code style
  take query and request parameters into account when matching routes
  mistake
  fix tests to run assertions on returned Crawler instances
  propagate groups to nested constraints

Author: nicolas-grekas

Tests/Console/Descriptor/AbstractDescriptorTest.php

@@ -302,7 +302,7 @@ private function getEventDispatcherDescriptionTestData(array $objects)
     }
 
     /** @dataProvider getDescribeContainerBuilderWithPriorityTagsTestData */
-    public function testDescribeContainerBuilderWithPriorityTags(ContainerBuilder $builder, $expectedDescription, array $options): void
+    public function testDescribeContainerBuilderWithPriorityTags(ContainerBuilder $builder, $expectedDescription, array $options)
     {
         $this->assertDescription($expectedDescription, $builder, $options);
     }

Tests/DependencyInjection/Fixtures/php/workflows.php

@@ -12,7 +12,7 @@
             'initial_marking' => ['draft'],
             'metadata' => [
                 'title' => 'article workflow',
-                'description' => 'workflow for articles'
+                'description' => 'workflow for articles',
             ],
             'places' => [
                 'draft',

Tests/Fixtures/ClassAliasExampleClass.php

@@ -4,7 +4,7 @@
 
 class_alias(
     ClassAliasTargetClass::class,
-    __NAMESPACE__ . '\ClassAliasExampleClass'
+    __NAMESPACE__.'\ClassAliasExampleClass'
 );
 
 if (false) {

Tests/Fixtures/Validation/Category.php

@@ -6,7 +6,7 @@
 
 class Category
 {
-    const NAME_PATTERN = '/\w+/';
+    public const NAME_PATTERN = '/\w+/';
 
     public $id;