Completed
Push — master ( 291e85...e418c2 )
by Guillaume
02:40
created
tests/Guesser/StatusGuesserTest.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,6 @@
 block discarded – undo
4 4
 
5 5
 use Hogosha\Monitor\Guesser\StatusGuesser;
6 6
 use Hogosha\Monitor\Model\Result;
7
-use Hogosha\Monitor\Model\ResultCollection;
8 7
 use Hogosha\Monitor\Model\UrlInfo;
9 8
 
10 9
 /**
Please login to merge, or discard this patch.
src/Runner/Runner.php 3 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
      * Constructor.
40 40
      *
41 41
      * @param UrlProvider  $urlProvider
42
-     * @param GuzzleClient $client
42
+     * @param Client $client
43 43
      */
44 44
     public function __construct(UrlProvider $urlProvider, Client $client)
45 45
     {
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
     /**
51 51
      * run.
52 52
      *
53
-     * @return array
53
+     * @return ResultCollection
54 54
      */
55 55
     public function run()
56 56
     {
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -61,9 +61,9 @@  discard block
 block discarded – undo
61 61
 
62 62
         $resultCollection = new ResultCollection();
63 63
 
64
-        $requests = function () use ($urls, $client, $resultCollection) {
64
+        $requests = function() use ($urls, $client, $resultCollection) {
65 65
             foreach ($urls as $url) {
66
-                yield function () use ($client, $url, $resultCollection) {
66
+                yield function() use ($client, $url, $resultCollection) {
67 67
                     return $client->sendAsync(
68 68
                         new Request(
69 69
                             $url->getMethod(),
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
                         ),
73 73
                         [
74 74
                             'timeout' => $url->getTimeout(),
75
-                            'on_stats' => function (TransferStats $tranferStats) use ($url, $resultCollection) {
75
+                            'on_stats' => function(TransferStats $tranferStats) use ($url, $resultCollection) {
76 76
 
77 77
                                 if ($tranferStats->hasResponse()) {
78 78
                                     $statusCode = $tranferStats->getResponse()->getStatusCode();
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,6 @@
 block discarded – undo
17 17
 
18 18
 use GuzzleHttp\Client;
19 19
 use GuzzleHttp\Pool;
20
-use GuzzleHttp\Promise;
21 20
 use GuzzleHttp\Psr7\Request;
22 21
 use GuzzleHttp\TransferStats;
23 22
 use Hogosha\Monitor\Client\GuzzleClient;
Please login to merge, or discard this patch.
src/Console/MonitorApplicationConfig.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
                 ->addOption('config', 'c', Option::OPTIONAL_VALUE, 'Config file path', rtrim(getcwd(), DIRECTORY_SEPARATOR))
58 58
                 ->beginCommand('init')
59 59
                     ->setDescription('Create a default configuration file if you do not have one')
60
-                    ->setHandler(function () use ($configurationLoader, $configurationDumper, $filesystem) {
60
+                    ->setHandler(function() use ($configurationLoader, $configurationDumper, $filesystem) {
61 61
                         return new InitHandler($configurationLoader, $configurationDumper, $filesystem);
62 62
                     })
63 63
                     ->setHelp('php <info>bin/monitor</info> init')
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
                 ->end()
66 66
                 ->beginCommand('run')
67 67
                     ->setDescription('Launch the monitor process')
68
-                    ->setHandler(function () use ($configurationLoader) {
68
+                    ->setHandler(function() use ($configurationLoader) {
69 69
                         return new RunHandler($configurationLoader);
70 70
                     })
71 71
                     ->setHelp('php <info>bin/monitor</info> run')
Please login to merge, or discard this patch.
src/Utils/Dom.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -107,7 +107,7 @@
 block discarded – undo
107 107
      *
108 108
      * @param string $element
109 109
      *
110
-     * @return DOMNodeList
110
+     * @return \DOMNodeList
111 111
      */
112 112
     public function xpath($element)
113 113
     {
Please login to merge, or discard this patch.
src/Validator/Validator.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -95,7 +95,7 @@
 block discarded – undo
95 95
      *
96 96
      * @param string $value
97 97
      *
98
-     * @return bool
98
+     * @return null|boolean
99 99
      */
100 100
     public function check($value)
101 101
     {
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -60,19 +60,19 @@
 block discarded – undo
60 60
             ]
61 61
         );
62 62
 
63
-        $resolver->setDefault('type', function (Options $options, $value) {
63
+        $resolver->setDefault('type', function(Options $options, $value) {
64 64
             if (null === $value) {
65 65
                 return;
66 66
             }
67 67
         });
68 68
 
69
-        $resolver->setDefault('match', function (Options $options, $value) {
69
+        $resolver->setDefault('match', function(Options $options, $value) {
70 70
             if (null === $value) {
71 71
                 return;
72 72
             }
73 73
         });
74 74
 
75
-        $resolver->setAllowedValues('type', function ($value) {
75
+        $resolver->setAllowedValues('type', function($value) {
76 76
             if (in_array($value, ['json', 'xml', 'html'])) {
77 77
                 return true;
78 78
             }
Please login to merge, or discard this patch.