Add typing

Author: tomzx

src/PHPSemVerChecker/Analyzer/Analyzer.php

@@ -14,7 +14,7 @@ class Analyzer
 	 * @param \PHPSemVerChecker\Registry\Registry $registryAfter
 	 * @return \PHPSemVerChecker\Report\Report
 	 */
-	public function analyze(Registry $registryBefore, Registry $registryAfter)
+	public function analyze(Registry $registryBefore, Registry $registryAfter): Report
 	{
 		$finalReport = new Report();
 

src/PHPSemVerChecker/Analyzer/ClassAnalyzer.php

@@ -20,7 +20,7 @@ class ClassAnalyzer
 	 * @param \PHPSemVerChecker\Registry\Registry $registryAfter
 	 * @return \PHPSemVerChecker\Report\Report
 	 */
-	public function analyze(Registry $registryBefore, Registry $registryAfter)
+	public function analyze(Registry $registryBefore, Registry $registryAfter): Report
 	{
 		$report = new Report();
 

src/PHPSemVerChecker/Analyzer/ClassMethodAnalyzer.php

@@ -36,11 +36,11 @@ class ClassMethodAnalyzer
 	protected $fileAfter;
 
 	/**
-	 * @param string $context
-	 * @param string $fileBefore
-	 * @param string $fileAfter
+	 * @param string      $context
+	 * @param string|null $fileBefore
+	 * @param string|null $fileAfter
 	 */
-	public function __construct($context, $fileBefore = null, $fileAfter = null)
+	public function __construct(string $context, string $fileBefore = null, string $fileAfter = null)
 	{
 		$this->context = $context;
 		$this->fileBefore = $fileBefore;
@@ -52,7 +52,7 @@ public function __construct($context, $fileBefore = null, $fileAfter = null)
 	 * @param \PhpParser\Node\Stmt $contextAfter
 	 * @return \PHPSemVerChecker\Report\Report
 	 */
-	public function analyze(Stmt $contextBefore, Stmt $contextAfter)
+	public function analyze(Stmt $contextBefore, Stmt $contextAfter): Report
 	{
 		$report = new Report();
 

src/PHPSemVerChecker/Analyzer/FunctionAnalyzer.php

@@ -9,7 +9,6 @@
 use PHPSemVerChecker\Operation\FunctionImplementationChanged;
 use PHPSemVerChecker\Operation\FunctionOperationUnary;
 use PHPSemVerChecker\Operation\FunctionParameterAdded;
-use PHPSemVerChecker\Operation\FunctionParameterChanged;
 use PHPSemVerChecker\Operation\FunctionParameterDefaultAdded;
 use PHPSemVerChecker\Operation\FunctionParameterDefaultRemoved;
 use PHPSemVerChecker\Operation\FunctionParameterDefaultValueChanged;
@@ -33,7 +32,7 @@ class FunctionAnalyzer
 	 * @param \PHPSemVerChecker\Registry\Registry $registryAfter
 	 * @return \PHPSemVerChecker\Report\Report
 	 */
-	public function analyze(Registry $registryBefore, Registry $registryAfter)
+	public function analyze(Registry $registryBefore, Registry $registryAfter): Report
 	{
 		$report = new Report();
 

src/PHPSemVerChecker/Analyzer/InterfaceAnalyzer.php

@@ -20,7 +20,7 @@ class InterfaceAnalyzer
 	 * @param \PHPSemVerChecker\Registry\Registry $registryAfter
 	 * @return \PHPSemVerChecker\Report\Report
 	 */
-	public function analyze(Registry $registryBefore, Registry $registryAfter)
+	public function analyze(Registry $registryBefore, Registry $registryAfter): Report
 	{
 		$report = new Report();
 

src/PHPSemVerChecker/Analyzer/PropertyAnalyzer.php

@@ -24,11 +24,11 @@ class PropertyAnalyzer
 	protected $fileAfter;
 
 	/**
-	 * @param string $context
-	 * @param string $fileBefore
-	 * @param string $fileAfter
+	 * @param string      $context
+	 * @param string|null $fileBefore
+	 * @param string|null $fileAfter
 	 */
-	public function __construct($context, $fileBefore = null, $fileAfter = null)
+	public function __construct(string $context, string $fileBefore = null, string $fileAfter = null)
 	{
 		$this->context = $context;
 		$this->fileBefore = $fileBefore;
@@ -40,7 +40,7 @@ public function __construct($context, $fileBefore = null, $fileAfter = null)
 	 * @param \PhpParser\Node\Stmt $contextAfter
 	 * @return \PHPSemVerChecker\Report\Report
 	 */
-	public function analyze(Stmt $contextBefore, Stmt $contextAfter)
+	public function analyze(Stmt $contextBefore, Stmt $contextAfter): Report
 	{
 		$report = new Report();
 
@@ -82,7 +82,7 @@ public function analyze(Stmt $contextBefore, Stmt $contextAfter)
 	 * @param \PhpParser\Node\Stmt $context
 	 * @return array
 	 */
-	protected function getProperties(Stmt $context)
+	protected function getProperties(Stmt $context): array
 	{
 		$properties = [];
 		foreach ($context->stmts as $stmt) {
@@ -97,7 +97,7 @@ protected function getProperties(Stmt $context)
 	 * @param \PhpParser\Node\Stmt\Property $property
 	 * @return string
 	 */
-	protected function getName(Property $property)
+	protected function getName(Property $property): string
 	{
 		return $property->props[0]->name->toString();
 	}

src/PHPSemVerChecker/Analyzer/TraitAnalyzer.php

@@ -20,7 +20,7 @@ class TraitAnalyzer
 	 * @param \PHPSemVerChecker\Registry\Registry $registryAfter
 	 * @return \PHPSemVerChecker\Report\Report
 	 */
-	public function analyze(Registry $registryBefore, Registry $registryAfter)
+	public function analyze(Registry $registryBefore, Registry $registryAfter): Report
 	{
 		$report = new Report();
 

src/PHPSemVerChecker/Comparator/Implementation.php

@@ -11,7 +11,7 @@ class Implementation
 	 * @param array $statementsB
 	 * @return bool
 	 */
-	public static function isSame(array $statementsA, array $statementsB)
+	public static function isSame(array $statementsA, array $statementsB): bool
 	{
 		// Naive way to check if two implementation are the same
 		$nodeDumper = new NodeDumper();

src/PHPSemVerChecker/Comparator/Node.php

@@ -9,7 +9,7 @@ class Node
 	 * @param \PhpParser\Node $nodeB
 	 * @return bool
 	 */
-	public static function isEqual(\PhpParser\Node $nodeA, \PhpParser\Node $nodeB)
+	public static function isEqual(\PhpParser\Node $nodeA, \PhpParser\Node $nodeB): bool
 	{
 		if ($nodeA->getType() !== $nodeB->getType()) {
 			return false;

src/PHPSemVerChecker/Comparator/Signature.php

@@ -9,7 +9,7 @@ class Signature
 	 * @param array $parametersB
 	 * @return array
 	 */
-	public static function analyze(array $parametersA, array $parametersB)
+	public static function analyze(array $parametersA, array $parametersB): array
 	{
 		$changes = [
 			'parameter_added'                 => false,