danielpieper / mrcli

src/Service/UserService.php

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
         }
         $users = $this->gitlabClient->users()->all($parameters);
 
-        return array_map(function ($user) {
+        return array_map(function($user) {
             return User::fromArray($user);
         }, $users);
     }

src/Service/ProjectService.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
     {
         $projects = $this->gitlabClient->users()->usersProjects($user->getId());
 
-        return array_map(function ($project) {
+        return array_map(function($project) {
             return Project::fromArray($project);
         }, $projects);
     }
@@ -72,7 +72,7 @@ 
             ]
         );
 
-        return array_map(function ($project) {
+        return array_map(function($project) {
             return Project::fromArray($project);
         }, $projects);
     }

src/Service/MergeRequestService.php

Spacing +2 added lines, -2 removed lines

@@ -86,7 +86,7 @@ 
             $projects[$project->getId()] = $project;
         }
 
-        return array_map(function ($mergeRequest) use ($projects) {
+        return array_map(function($mergeRequest) use ($projects) {
             return $this->transform($projects[$mergeRequest['project_id']], $mergeRequest);
         }, $mergeRequests);
     }
@@ -119,7 +119,7 @@ 
             ]
         );
 
-        return array_map(function ($mergeRequest) use ($project) {
+        return array_map(function($mergeRequest) use ($project) {
             return $this->transform($project, $mergeRequest);
         }, $mergeRequests);
     }

src/Service/MergeRequestApprovalService.php

Spacing +4 added lines, -4 removed lines

@@ -65,17 +65,17 @@ 
         $this->transformApprovers($mergeRequestApproval['approvers']);
         $this->transformApprovers($mergeRequestApproval['suggested_approvers']);
 
-        $approverIdList = array_map(function (User $approver) {
+        $approverIdList = array_map(function(User $approver) {
             return $approver->getId();
         }, $mergeRequestApproval['approvers']);
         $mergeRequestApproval['suggested_approvers'] = array_filter(
             $mergeRequestApproval['suggested_approvers'],
-            function (User $suggestedApprover) use ($approverIdList) {
+            function(User $suggestedApprover) use ($approverIdList) {
                 return !in_array($suggestedApprover->getId(), $approverIdList);
             }
         );
 
-        array_walk($mergeRequestApproval['approver_groups'], function (&$approverGroup) {
+        array_walk($mergeRequestApproval['approver_groups'], function(&$approverGroup) {
             $approverGroup = Group::fromArray($approverGroup['group']);
         });
 
@@ -88,7 +88,7 @@ 
      */
     private function transformApprovers(array &$approvers): void
     {
-        array_walk($approvers, function (&$approver) {
+        array_walk($approvers, function(&$approver) {
             // suggested_approvers are not encapsulated in 'user'...
             if (isset($approver['user'])) {
                 $approver = $approver['user'];

src/Command/ProjectCommand.php

Spacing +3 added lines, -3 removed lines

@@ -68,7 +68,7 @@ 
     private function getProjects(array $names): array
     {
         $projects = $this->projectService->all();
-        $projects = array_filter($projects, function (Project $project) use ($names) {
+        $projects = array_filter($projects, function(Project $project) use ($names) {
             return in_array($project->getName(), $names);
         });
         if (count($projects) == 0) {
@@ -105,7 +105,7 @@ 
         foreach ($mergeRequests as $mergeRequest) {
             $mergeRequestApprovals[] = $this->mergeRequestApprovalService->get($mergeRequest);
         }
-        $mergeRequestApprovals = array_filter($mergeRequestApprovals, function (MergeRequestApproval $item) {
+        $mergeRequestApprovals = array_filter($mergeRequestApprovals, function(MergeRequestApproval $item) {
             $hasApprovalsLeft = $item->getApprovalsLeft() > 0;
             $isWorkInProgress = $item->getMergeRequest()->isWorkInProgress();
 
@@ -130,7 +130,7 @@ 
         $mergeRequests = $this->getMergeRequests($projects);
         $mergeRequestApprovals = $this->getMergeRequestApprovals($mergeRequests);
 
-        usort($mergeRequestApprovals, function (MergeRequestApproval $approvalA, MergeRequestApproval $approvalB) {
+        usort($mergeRequestApprovals, function(MergeRequestApproval $approvalA, MergeRequestApproval $approvalB) {
             if ($approvalA->getCreatedAt()->equalTo($approvalB->getCreatedAt())) {
                 return 0;
             }

src/Command/OverviewCommand.php

Spacing +4 added lines, -4 removed lines

@@ -47,7 +47,7 @@ 
         foreach ($mergeRequests as $mergeRequest) {
             $mergeRequestApprovals[] = $this->mergeRequestApprovalService->get($mergeRequest);
         }
-        return array_filter($mergeRequestApprovals, function (MergeRequestApproval $item) {
+        return array_filter($mergeRequestApprovals, function(MergeRequestApproval $item) {
             $hasApprovalsLeft = $item->getApprovalsLeft() > 0;
             $isWorkInProgress = $item->getMergeRequest()->isWorkInProgress();
 
@@ -93,7 +93,7 @@ 
      */
     private function getHeaders(array $mergeRequestApprovals): array
     {
-        $headers = array_unique(array_map(function (MergeRequestApproval $mergeRequestApproval) {
+        $headers = array_unique(array_map(function(MergeRequestApproval $mergeRequestApproval) {
             return $mergeRequestApproval->getMergeRequest()->getProject()->getName();
         }, $mergeRequestApprovals));
         array_unshift($headers, 'Total');
@@ -112,7 +112,7 @@ 
         $headerIndices = array_flip($headers);
         $rows = array_reduce(
             $mergeRequestApprovals,
-            function ($carry, MergeRequestApproval $mergeRequestApproval) use ($headerIndices) {
+            function($carry, MergeRequestApproval $mergeRequestApproval) use ($headerIndices) {
                 $approverNames = $mergeRequestApproval->getApproverNames();
                 foreach ($approverNames as $approverName) {
                     if (!isset($carry[$approverName])) {
@@ -130,7 +130,7 @@ 
             []
         );
 
-        usort($rows, function ($approverA, $approverB) {
+        usort($rows, function($approverA, $approverB) {
             if ($approverA[1] == $approverB[1]) {
                 return 0;
             }

src/Command/ApproverCommand.php

Spacing +3 added lines, -3 removed lines

@@ -110,13 +110,13 @@ 
         }
         $mergeRequestApprovals = array_filter(
             $mergeRequestApprovals,
-            function (MergeRequestApproval $item) use ($user, $includeSuggestedApprovers) {
+            function(MergeRequestApproval $item) use ($user, $includeSuggestedApprovers) {
                 $hasApprovalsLeft = $item->getApprovalsLeft() > 0;
                 $isWorkInProgress = $item->getMergeRequest()->isWorkInProgress();
                 $isApprover = in_array($user->getUsername(), $item->getApproverNames());
                 $isAssignee = $item->getMergeRequest()->getAssignee() == $user->getUsername();
                 $isSuggestedApprover = $includeSuggestedApprovers
-                    && in_array($user->getUsername(), $item->getSuggestedApproverNames()) ;
+                    && in_array($user->getUsername(), $item->getSuggestedApproverNames());
 
                 return $hasApprovalsLeft && !$isWorkInProgress && ($isApprover || $isAssignee || $isSuggestedApprover);
             }
@@ -143,7 +143,7 @@ 
             $input->getOption('include-suggested')
         );
 
-        usort($mergeRequestApprovals, function (MergeRequestApproval $approvalA, MergeRequestApproval $approvalB) {
+        usort($mergeRequestApprovals, function(MergeRequestApproval $approvalA, MergeRequestApproval $approvalB) {
             if ($approvalA->getCreatedAt()->equalTo($approvalB->getCreatedAt())) {
                 return 0;
             }