Skip to content

Commit 72c3895

Browse files
committed
fixup! misc
1 parent b9ed48e commit 72c3895

File tree

6 files changed

+43
-36
lines changed

6 files changed

+43
-36
lines changed

src/Command/AnalyseCommand.php

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
use Rector\Monitor\RuleProcessor\MetafileProcessor\NoPHPStanBaselineMetafileProcessor;
1313
use Rector\Monitor\RuleProcessor\MissingPackagesRuleProcessor;
1414
use Rector\Monitor\RuleProcessor\TooLowPackagesRulesProcessor;
15-
use Rector\Monitor\ValueObject\Repository;
1615
use Rector\Monitor\ValueObject\RepositoryCollection;
1716
use Symfony\Component\Console\Command\Command;
1817
use Symfony\Component\Console\Input\InputInterface;
@@ -65,7 +64,10 @@ protected function execute(InputInterface $input, OutputInterface $output): int
6564
$monitorConfig = $this->monitorConfigProvider->provide();
6665
$shouldClearCache = (bool) $input->getOption('clear-cache');
6766

68-
$this->repositoryMetafilesResolver->decorateRepositories($monitorConfig->getRepositoryCollection(), $shouldClearCache);
67+
$this->repositoryMetafilesResolver->decorateRepositories(
68+
$monitorConfig->getRepositoryCollection(),
69+
$shouldClearCache
70+
);
6971

7072
$this->symfonyStyle->title('Repository analysis report');
7173

@@ -76,7 +78,9 @@ protected function execute(InputInterface $input, OutputInterface $output): int
7678

7779
$composerJson = $repository->getComposerJson();
7880

79-
$this->symfonyStyle->writeln(sprintf('<fg=yellow;options=bold>%d) %s</>', $key+1, $composerJson->getRepositoryName()));
81+
$this->symfonyStyle->writeln(
82+
sprintf('<fg=yellow;options=bold>%d) %s</>', $key + 1, $composerJson->getRepositoryName())
83+
);
8084
foreach ($this->ruleProcessors as $ruleProcessor) {
8185
$ruleProcessor->process($monitorConfig, $composerJson, $errorCollector);
8286
}

src/Command/MatrixCommand.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,8 @@ protected function execute(InputInterface $input, OutputInterface $output): int
5555
$this->symfonyStyle->title('Showing Dependency Matrix');
5656

5757
$this->repositoryMetafilesResolver->decorateRepositories(
58-
$monitorConfig->getRepositoryCollection(), $shouldClearCache
58+
$monitorConfig->getRepositoryCollection(),
59+
$shouldClearCache
5960
);
6061

6162
$repositoryCollection = $monitorConfig->getRepositoryCollection();

src/Config/MonitorConfig.php

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -60,28 +60,28 @@ public function addRepositories(array $repositories): self
6060
return $this;
6161
}
6262

63-
// /**
64-
// * @return Repository[]
65-
// */
66-
// private function getRepositories(): RepositoryCollection
67-
// {
68-
// if ($this->repositoryCollection instanceof RepositoryCollection) {
69-
// return $this->repositoryCollection;
70-
// }
71-
//
72-
// $repositories = [];
73-
// foreach ($this->repositories as $repository) {
74-
// $repositories[] = new Repository($repository);
75-
// }
76-
//
77-
// foreach ($this->repositoriesWithBranches as $repository => $branch) {
78-
// $repositories[] = new Repository($repository, $branch);
79-
// }
80-
//
81-
// $this->repositoryCollection = new RepositoryCollection($repositories);
82-
//
83-
// return $repositories;
84-
// }
63+
// /**
64+
// * @return Repository[]
65+
// */
66+
// private function getRepositories(): RepositoryCollection
67+
// {
68+
// if ($this->repositoryCollection instanceof RepositoryCollection) {
69+
// return $this->repositoryCollection;
70+
// }
71+
//
72+
// $repositories = [];
73+
// foreach ($this->repositories as $repository) {
74+
// $repositories[] = new Repository($repository);
75+
// }
76+
//
77+
// foreach ($this->repositoriesWithBranches as $repository => $branch) {
78+
// $repositories[] = new Repository($repository, $branch);
79+
// }
80+
//
81+
// $this->repositoryCollection = new RepositoryCollection($repositories);
82+
//
83+
// return $repositories;
84+
// }
8585

8686
/**
8787
* @param string[] $packageNames

src/Config/MonitorConfigProvider.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@ public function provide(): MonitorConfig
2525

2626
// basic validation
2727
Assert::notEmpty(
28-
$monitorConfig->getRepositoryCollection()->all(),
28+
$monitorConfig->getRepositoryCollection()
29+
->all(),
2930
'At least one repository is required. Add it in "monitor.php" config via "->addRepository(...)" method'
3031
);
3132

src/Git/RepositoryMetafilesResolver.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,8 @@ public function decorateRepositories(RepositoryCollection $repositoryCollection,
4242

4343
$this->symfonyStyle->writeln(sprintf(
4444
' * loading "<fg=green>%s</>" repository from cache',
45-
$repository->getComposerJson()->getRepositoryName()
45+
$repository->getComposerJson()
46+
->getRepositoryName()
4647
));
4748

4849
continue;

src/ValueObject/RepositoryCollection.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,14 @@ public function allSorterByPackageCount(): array
7575
return $composerJsons;
7676
}
7777

78+
/**
79+
* @return Repository[]
80+
*/
81+
public function all(): array
82+
{
83+
return $this->repositories;
84+
}
85+
7886
/**
7987
* @param string[] $uniquePackageNames
8088
* @return string[]
@@ -86,12 +94,4 @@ private function filterOutExtensions(array $uniquePackageNames): array
8694
fn (string $packageName): bool => ! str_starts_with($packageName, 'ext-')
8795
);
8896
}
89-
90-
/**
91-
* @return Repository[]
92-
*/
93-
public function all(): array
94-
{
95-
return $this->repositories;
96-
}
9797
}

0 commit comments

Comments
 (0)