Merge pull request #595 from greg0ire/internalize

Make classes internal

Author: greg0ire

UPGRADE.md

@@ -2,10 +2,10 @@
 
 ## 4.0.0
 
-## BC break: final classes
+## BC break: final, internal classes
 
-Some classes have been marked as `final` because they are not supposed to be
-extended. Consider using composition instead of inheritance.
+Some classes have been marked as `final` and `@internal` because they are not
+supposed to be extended or even referenced outside of the bundle.
 
 ### BC break: type declarations
 

src/Collector/MigrationsCollector.php

@@ -16,6 +16,7 @@
 use function count;
 use function get_class;
 
+/** @internal */
 final class MigrationsCollector extends DataCollector
 {
     public function __construct(

src/Collector/MigrationsFlattener.php

@@ -13,6 +13,7 @@
 
 use function array_map;
 
+/** @internal */
 final class MigrationsFlattener
 {
     /**

src/DependencyInjection/CompilerPass/ConfigureDependencyFactoryPass.php

@@ -19,6 +19,7 @@
 use function is_string;
 use function sprintf;
 
+/** @internal */
 final class ConfigureDependencyFactoryPass implements CompilerPassInterface
 {
     public function process(ContainerBuilder $container): void

src/DependencyInjection/Configuration.php

@@ -19,6 +19,7 @@
 use function strtoupper;
 use function substr;
 
+/** @internal */
 final class Configuration implements ConfigurationInterface
 {
     public function getConfigTreeBuilder(): TreeBuilder

src/DependencyInjection/DoctrineMigrationsExtension.php

@@ -28,6 +28,7 @@
 use function strlen;
 use function substr;
 
+/** @internal */
 final class DoctrineMigrationsExtension extends Extension
 {
     /**

src/EventListener/SchemaFilterListener.php

@@ -13,6 +13,8 @@
  * Acts as a schema filter that hides the migration metadata table except
  * when the execution context is that of command inside the migrations
  * namespace.
+ *
+ * @internal
  */
 final class SchemaFilterListener
 {