Remove cron schedule from run-tests.yml

Removed scheduled cron job from workflow.

.github/workflows/run-tests.yml

@@ -7,8 +7,6 @@ on:
   pull_request:
     branches:
     - "*"
-  schedule:
-  - cron: '0 0 * * *'
 
 jobs:
   php-tests:

src/Barryvdh/Reflection/DocBlock/Serializer.php

@@ -206,8 +206,9 @@ class Serializer
             $text = wordwrap($text, $wrapLength);
         }
         $text = str_replace("\n", "\n{$indent} * ", $text);
+        $text = preg_replace('/^(\s*\*)[ \t]+$/m', '$1', $text);
 
-        $comment = "{$firstIndent}/**\n{$indent} * {$text}\n{$indent} *\n";
+        $comment = !empty($text)? "{$firstIndent}/**\n{$indent} * {$text}\n{$indent} *\n" : "{$firstIndent}/**\n";
 
         $tags = array_values($docblock->getTags());
 
@@ -220,6 +221,7 @@ class Serializer
                 $tagText = wordwrap($tagText, $wrapLength);
             }
             $tagText = str_replace("\n", "\n{$indent} * ", $tagText);
+            $tagText = preg_replace('/^(\s*\*)[ \t]+$/m', '$1', $tagText);
 
             $comment .= "{$indent} * {$tagText}\n";
 

src/Barryvdh/Reflection/DocBlock/Tag.php

@@ -409,6 +409,6 @@ class Tag implements \Reflector
      */
     public function __toString()
     {
-        return "@{$this->getName()} {$this->getContent()}";
+        return trim("@{$this->getName()} {$this->getContent()}");
     }
 }

src/Barryvdh/Reflection/DocBlock/Tag/ParamTag.php

@@ -54,16 +54,15 @@ class ParamTag extends ReturnTag
         for($pos = 0, $stacks = []; $pos < strlen($rest); $pos++) {
             $char = $rest[$pos];
 
-            if($char === '<') {
+            if(in_array($char, ['<', '(', '[', '{'])) {
                 array_unshift($stacks, $char);
             }
-            if($char === '(') {
+            if(
-                array_unshift($stacks, $char);
+                ($char === '>' && isset($stacks[0]) && $stacks[0] === '<')
-            }
+                || ($char === ')' && isset($stacks[0]) && $stacks[0] === '(')
-            if($char === '>' && isset($stacks[0]) && $stacks[0] === '<') {
+                || ($char === ']' && isset($stacks[0]) && $stacks[0] === '[')
-                array_shift($stacks);
+                || ($char === '}' && isset($stacks[0]) && $stacks[0] === '{')
-            }
+            ) {
-            if($char === ')' && isset($stacks[0]) && $stacks[0] === '(') {
                 array_shift($stacks);
             }
 

src/Barryvdh/Reflection/DocBlock/Type/Collection.php

@@ -161,9 +161,9 @@ class Collection extends \ArrayObject
                 $type_parts[] = $curr_type;
                 $curr_type = '';
             } else {
-                if ($char === '<' || $char === '(') {
+                if (in_array($char, ['<', '(', '[', '{'])) {
                     $nest_level++;
-                } else if ($char === '>' || $char === ')') {
+                } else if (in_array($char, ['>', ')', ']', '}'])) {
                     $nest_level--;
                 }
 
@@ -201,7 +201,13 @@ class Collection extends \ArrayObject
             return '';
         }
 
-        if (preg_match('/^[\w-]+<.*>$/', $type)) {
+        // Check for generics values and array shapes
+        if (preg_match('/^[\w-]+(<.+>|\[.+\]|{.+})$/', $type)) {
+            return $type;
+        }
+
+        // Check for callable types
+        if (preg_match('/\(.*?(?=\:)/', $type)) {
             return $type;
         }
 
@@ -209,6 +215,11 @@ class Collection extends \ArrayObject
             return $type;
         }
 
+        // Literal strings
+        if ($type[0] === '"' || $type[0] === "'") {
+            return $type;
+        }
+
         if ($this->isTypeAnArray($type)) {
             return $this->expand(substr($type, 0, -2)) . self::OPERATOR_ARRAY;
         }

tests/Barryvdh/Reflection/DocBlock/Tag/ParamTagTest.php

@@ -183,6 +183,24 @@ class ParamTagTest extends TestCase
                 '$callback',
                 ''
             ),
+
+            // array shapes
+            array(
+                'param',
+                'array{foo: string, bar: int} $array',
+                'array{foo: string, bar: int}',
+                array('array{foo: string, bar: int}'),
+                '$array',
+                ''
+            ),
+            array(
+                'param',
+                'MyArray[\'key\'] $value',
+                'MyArray[\'key\']',
+                array('MyArray[\'key\']'),
+                '$value',
+                ''
+            )
         );
     }
 }

tests/Barryvdh/Reflection/DocBlock/Tag/ReturnTagTest.php

@@ -119,6 +119,13 @@ class ReturnTagTest extends TestCase
                 'array<int, string|bool>|string',
                 array('array<int, string|bool>', 'string'),
                 'Types of Bobs'
+            ),
+            array(
+                'return',
+                'MyArray[\'key\'] Type of Bobs',
+                'MyArray[\'key\']',
+                array('MyArray[\'key\']'),
+                'Type of Bobs'
             )
         );
     }

tests/Barryvdh/Reflection/DocBlock/Type/CollectionTest.php

@@ -239,6 +239,18 @@ class CollectionTest extends TestCase
                 'array<int, string|array<int, bool>>|array<int, float>|string',
                 array('array<int, string|array<int, bool>>', 'array<int, float>', 'string')
             ),
+            array(
+                'array{ 0: string, 1: string|int }',
+                array('array{ 0: string, 1: string|int }')
+            ),
+            array(
+                "array{ 'key': string, 'value': string|int }",
+                array("array{ 'key': string, 'value': string|int }")
+            ),
+            array(
+                "MyArray['bar']",
+                array("MyArray['bar']")
+            ),
             array(
                 'LinkDescriptor::setLink()',
                 array($namespace.'LinkDescriptor::setLink()')
@@ -283,6 +295,18 @@ class CollectionTest extends TestCase
                 'iterable<string>',
                 array('iterable<string>')
             ),
+            array(
+                'callable',
+                array('callable')
+            ),
+            array(
+                'callable(int, string): int',
+                array('callable(int, string): int')
+            ),
+            array(
+                "'text'",
+                array("'text'")
+            )
         );
     }