From 73b9f09dbc0e7edcd4d0bf498e8a7d207636dc63 Mon Sep 17 00:00:00 2001 From: Tony Hegyes Date: Thu, 29 Oct 2026 10:56:35 +0100 Subject: [PATCH] Drop the redundant public from the plugin's set-only properties Plugin declared $migrations and $kernel as public protected(set), while a set-only visibility already reads as public. The framework packages write protected(set) alone, and the style guide's rule covers plugin code too, so the template now matches what every plugin copies from it. Assisted-by: Claude Code:claude-opus-5-5 --- src/Plugin.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Plugin.php b/src/Plugin.php index 96d49c3..8a8da5f 100644 --- a/src/Plugin.php +++ b/src/Plugin.php @@ -88,7 +88,7 @@ final class Plugin { * * @var MigrationRunner|null */ - public protected(set) ?MigrationRunner $migrations = null; + protected(set) ?MigrationRunner $migrations = null; /** * The kernel, once the composition built it, which tells which components loaded and which were gated off. @@ -98,7 +98,7 @@ final class Plugin { * * @var PluginKernel|null */ - public protected(set) ?PluginKernel $kernel = null; + protected(set) ?PluginKernel $kernel = null; /** * The logger every part of the plugin writes to.