Merge pull request #2 from cspray/adr/deprecate-configuration

Deprecate Configuration and ConfigurationAttribute

Author: cspray

composer.json

@@ -4,7 +4,7 @@
   "license": ["MIT"],
   "require": {
     "php": "^8.1",
-    "cspray/annotated-container-adr": "^1.0"
+    "cspray/annotated-container-adr": "^2"
   },
   "autoload": {
     "psr-4": {

composer.lock

@@ -4,15 +4,21 @@
 
 use Attribute;
 use Cspray\AnnotatedContainer\ArchitecturalDecisionRecords\ConfigurationCannotBeAssignedProfiles;
+use Cspray\AnnotatedContainer\ArchitecturalDecisionRecords\DeprecateConfigurationInFavorOfCustomServiceAttribute;
+use JetBrains\PhpStorm\Deprecated;
 
 /**
  * An Attribute that defines a class as being a Configuration object.
  *
  * A Configuration object is expected to consist primarily of public readonly properties that have been annotated with
  * the Inject Attribute.
  */
-#[ConfigurationCannotBeAssignedProfiles]
-#[Attribute(Attribute::TARGET_CLASS)]
+#[
+    ConfigurationCannotBeAssignedProfiles,
+    DeprecateConfigurationInFavorOfCustomServiceAttribute,
+    Deprecated('Please see DeprecateConfigurationInFavorOfCustomServiceAttribute ADR'),
+    Attribute(Attribute::TARGET_CLASS)
+]
 final class Configuration implements ConfigurationAttribute {
 
     /**

src/Configuration.php

@@ -2,12 +2,19 @@
 
 namespace Cspray\AnnotatedContainer\Attribute;
 
+use Cspray\AnnotatedContainer\ArchitecturalDecisionRecords\DeprecateConfigurationInFavorOfCustomServiceAttribute;
+use JetBrains\PhpStorm\Deprecated;
+
 /**
  * Can be implemented by your own classes to act as an Attribute for defining a Configuration.
  *
  * The class that implements this interface should also be marked as an Attribute; it should be non-repeatable and only
  * able to target the class construct.
  */
+#[
+    DeprecateConfigurationInFavorOfCustomServiceAttribute,
+    Deprecated('Please see DeprecateConfigurationInFavorOfCustomServiceAttribute ADR')
+]
 interface ConfigurationAttribute {
 
     /**