minor #20102 minor : the process method from CompilerPassInterface should be public (magnetik)

xabbuh · xabbuh · commit 86acc2a5ff7d · 2024-08-08T13:33:18.000+02:00
This PR was merged into the 5.4 branch. Discussion ---------- minor : the process method from CompilerPassInterface should be public Since the method is defined in CompilerPassInterface as public, it should be defined as public in the Kernel. Commits ------- f206ee7 minor : the process method from CompilerPassInterface should be public
diff --git a/testing.rst b/testing.rst
@@ -651,7 +651,7 @@ to remove the ``kernel.reset`` tag from some services in your test environment::
 
         // ...
 
-        protected function process(ContainerBuilder $container): void
+        public function process(ContainerBuilder $container): void
         {
             if ('test' === $this->environment) {
                 // prevents the security token to be cleared

Original file line number	Diff line number	Diff line change
@@ -651,7 +651,7 @@ to remove the ``kernel.reset`` tag from some services in your test environment::
`651`	`651`
`652`	`652`	`// ...`
`653`	`653`
`654`		`- protected function process(ContainerBuilder $container): void`
	`654`	`+ public function process(ContainerBuilder $container): void`
`655`	`655`	`{`
`656`	`656`	`if ('test' === $this->environment) {`
`657`	`657`	`// prevents the security token to be cleared`