Issue #3293157 by phenaproxima, tedbow: Make everything internal

package_manager/src/EventSubscriber/UpdateDataSubscriber.php

@@ -8,8 +8,13 @@ use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 
 /**
  * Clears stale update data once staged changes have been applied.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class UpdateDataSubscriber implements EventSubscriberInterface {
+final class UpdateDataSubscriber implements EventSubscriberInterface {
 
   /**
    * The update manager service.

package_manager/src/ExecutableFinder.php

@@ -9,8 +9,13 @@ use Symfony\Component\Process\ExecutableFinder as SymfonyExecutableFinder;
 
 /**
  * An executable finder which looks for executable paths in configuration.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class ExecutableFinder implements ExecutableFinderInterface {
+final class ExecutableFinder implements ExecutableFinderInterface {
 
   /**
    * The decorated executable finder.

package_manager/src/FileSyncerFactory.php

@@ -11,8 +11,13 @@ use Symfony\Component\Process\ExecutableFinder;
 
 /**
  * A file syncer factory which creates a file syncer according to configuration.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class FileSyncerFactory {
+final class FileSyncerFactory {
 
   /**
    * The decorated file syncer factory.

package_manager/src/PackageManagerServiceProvider.php

@@ -9,8 +9,13 @@ use Symfony\Component\DependencyInjection\Reference;
 
 /**
  * Defines dynamic container services for Package Manager.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class PackageManagerServiceProvider extends ServiceProviderBase {
+final class PackageManagerServiceProvider extends ServiceProviderBase {
 
   /**
    * {@inheritdoc}

package_manager/src/PackageManagerUninstallValidator.php

@@ -9,8 +9,13 @@ use Symfony\Component\DependencyInjection\ContainerAwareTrait;
 
 /**
  * Prevents any module from being uninstalled if update is in process.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class PackageManagerUninstallValidator implements ModuleUninstallValidatorInterface, ContainerAwareInterface {
+final class PackageManagerUninstallValidator implements ModuleUninstallValidatorInterface, ContainerAwareInterface {
 
   use ContainerAwareTrait;
   use StringTranslationTrait;

package_manager/src/PathExcluder/GitExcluder.php

@@ -11,8 +11,13 @@ use Symfony\Component\Finder\Finder;
 
 /**
  * Excludes .git directories from staging operations.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class GitExcluder implements EventSubscriberInterface {
+final class GitExcluder implements EventSubscriberInterface {
 
   use PathExclusionsTrait;
 

package_manager/src/PathExcluder/SiteConfigurationExcluder.php

@@ -10,6 +10,11 @@ use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 
 /**
  * Excludes site configuration files from staging areas.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
 class SiteConfigurationExcluder implements EventSubscriberInterface {
 

package_manager/src/PathExcluder/SiteFilesExcluder.php

@@ -13,8 +13,13 @@ use Symfony\Component\Filesystem\Filesystem;
 
 /**
  * Excludes public and private files from staging operations.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class SiteFilesExcluder implements EventSubscriberInterface {
+final class SiteFilesExcluder implements EventSubscriberInterface {
 
   use PathExclusionsTrait;
 

package_manager/src/PathExcluder/SqliteDatabaseExcluder.php

@@ -11,6 +11,11 @@ use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 
 /**
  * Excludes SQLite database files from staging operations.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
 class SqliteDatabaseExcluder implements EventSubscriberInterface {
 

package_manager/src/PathExcluder/TestSiteExcluder.php

@@ -10,8 +10,13 @@ use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 
 /**
  * Excludes 'sites/simpletest' from staging operations.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class TestSiteExcluder implements EventSubscriberInterface {
+final class TestSiteExcluder implements EventSubscriberInterface {
 
   use PathExclusionsTrait;
 

package_manager/src/PathExcluder/VendorHardeningExcluder.php

@@ -10,8 +10,13 @@ use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 
 /**
  * Excludes vendor hardening files from staging operations.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class VendorHardeningExcluder implements EventSubscriberInterface {
+final class VendorHardeningExcluder implements EventSubscriberInterface {
 
   use PathExclusionsTrait;
 

package_manager/src/ProcessFactory.php

@@ -10,6 +10,11 @@ use Symfony\Component\Process\Process;
 
 /**
  * Defines a process factory which sets the COMPOSER_HOME environment variable.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
 final class ProcessFactory implements ProcessFactoryInterface {
 

package_manager/src/ValidationResult.php

@@ -8,7 +8,7 @@ use Drupal\system\SystemManager;
 /**
  * A value object to contain the results of a validation.
  */
-class ValidationResult {
+final class ValidationResult {
 
   /**
    * A succinct summary of the results.

package_manager/src/Validator/ComposerExecutableValidator.php

@@ -15,8 +15,13 @@ use PhpTuf\ComposerStager\Domain\Service\ProcessRunner\ComposerRunnerInterface;
 
 /**
  * Validates the Composer executable is the correct version.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class ComposerExecutableValidator implements PreOperationStageValidatorInterface, ProcessOutputCallbackInterface {
+final class ComposerExecutableValidator implements PreOperationStageValidatorInterface, ProcessOutputCallbackInterface {
 
   use StringTranslationTrait;
 

package_manager/src/Validator/ComposerSettingsValidator.php

@@ -9,8 +9,13 @@ use Drupal\package_manager\Event\PreOperationStageEvent;
 
 /**
  * Validates certain Composer settings.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class ComposerSettingsValidator implements PreOperationStageValidatorInterface {
+final class ComposerSettingsValidator implements PreOperationStageValidatorInterface {
 
   use StringTranslationTrait;
 

package_manager/src/Validator/DiskSpaceValidator.php

@@ -12,6 +12,11 @@ use Drupal\package_manager\PathLocator;
 
 /**
  * Validates that there is enough free disk space to do staging operations.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
 class DiskSpaceValidator implements PreOperationStageValidatorInterface {
 

package_manager/src/Validator/LockFileValidator.php

@@ -14,8 +14,13 @@ use Drupal\package_manager\PathLocator;
 
 /**
  * Checks that the active lock file is unchanged during stage operations.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class LockFileValidator implements PreOperationStageValidatorInterface {
+final class LockFileValidator implements PreOperationStageValidatorInterface {
 
   use StringTranslationTrait;
 

package_manager/src/Validator/MultisiteValidator.php

@@ -10,8 +10,13 @@ use Drupal\package_manager\PathLocator;
 
 /**
  * Checks that the current site is not part of a multisite.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class MultisiteValidator implements PreOperationStageValidatorInterface {
+final class MultisiteValidator implements PreOperationStageValidatorInterface {
 
   use StringTranslationTrait;
 

package_manager/src/Validator/PendingUpdatesValidator.php

@@ -11,8 +11,13 @@ use Drupal\Core\Url;
 
 /**
  * Validates that there are no pending database updates.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class PendingUpdatesValidator implements PreOperationStageValidatorInterface {
+final class PendingUpdatesValidator implements PreOperationStageValidatorInterface {
 
   use StringTranslationTrait;
 

package_manager/src/Validator/SettingsValidator.php

@@ -10,8 +10,13 @@ use Drupal\package_manager\Event\PreOperationStageEvent;
 
 /**
  * Checks that Drupal's settings are valid for Package Manager.
+ *
+ * @internal
+ *   This is an internal part of Package Manager and may be changed or removed
+ *   at any time without warning. External code should not interact with this
+ *   class.
  */
-class SettingsValidator implements PreOperationStageValidatorInterface {
+final class SettingsValidator implements PreOperationStageValidatorInterface {
 
   use StringTranslationTrait;