[BCB] Parameter $context of NodeScopeResolver::processStmtNodes() made required

Author: ondrejmirtes

UPGRADING.md

@@ -51,3 +51,4 @@ This method now longer accepts `Expr $rootExpr`. If you want to change it, call
 ### Minor backward compatibility breaks
 
 * Parameter `$callableParameters` of [`MutatingScope::enterAnonymousFunction()`](https://apiref.phpstan.org/2.0.x/PHPStan.Analyser.MutatingScope.html#_enterAnonymousFunction) and [`enterArrowFunction()`](https://apiref.phpstan.org/2.0.x/PHPStan.Analyser.MutatingScope.html#_enterArrowFunction) made required
+* Parameter `StatementContext $context` of [`NodeScopeResolver::processStmtNodes()`](https://apiref.phpstan.org/2.0.x/PHPStan.Analyser.NodeScopeResolver.html#_processStmtNodes) made required

src/Analyser/NodeScopeResolver.php

@@ -329,12 +329,9 @@ public function processStmtNodes(
 		array $stmts,
 		MutatingScope $scope,
 		callable $nodeCallback,
-		?StatementContext $context = null,
+		StatementContext $context,
 	): StatementResult
 	{
-		if ($context === null) {
-			$context = StatementContext::createTopLevel();
-		}
 		$exitPoints = [];
 		$throwPoints = [];
 		$impurePoints = [];

tests/PHPStan/Analyser/StatementResultTest.php

@@ -406,6 +406,7 @@ public function testIsAlwaysTerminating(
 			$scope,
 			static function (): void {
 			},
+			StatementContext::createTopLevel(),
 		);
 		$this->assertSame($expectedIsAlwaysTerminating, $result->isAlwaysTerminating());
 	}