fix: add #[\Override] attributes to interface implementations

Add Override attributes to methods implementing:
- ResourceDetectorInterface (getResource)
- SamplerInterface (shouldSample, getDescription)
- TextMapPropagatorInterface (fields, inject, extract)
- PropagationGetterInterface (keys, get)
- PropagationSetterInterface (set)
- InstrumentationInterface (multiple methods)
- RuleSetInterface (samplingRules, delegate)

Author: cedricziel

src/Aws/src/AwsSdkInstrumentation.php

@@ -37,46 +37,55 @@ class AwsSdkInstrumentation implements InstrumentationInterface
 
     private array $spanStorage = [];
 
+    #[\Override]
     public function getName(): string
     {
         return self::NAME;
     }
 
+    #[\Override]
     public function getVersion(): ?string
     {
         return self::VERSION;
     }
 
+    #[\Override]
     public function getSchemaUrl(): ?string
     {
         return null;
     }
 
+    #[\Override]
     public function init(): bool
     {
         return true;
     }
 
+    #[\Override]
     public function setPropagator(TextMapPropagatorInterface $propagator): void
     {
         $this->propagator = $propagator;
     }
 
+    #[\Override]
     public function getPropagator(): TextMapPropagatorInterface
     {
         return $this->propagator;
     }
 
+    #[\Override]
     public function setTracerProvider(TracerProviderInterface $tracerProvider): void
     {
         $this->tracerProvider = $tracerProvider;
     }
 
+    #[\Override]
     public function getTracerProvider(): TracerProviderInterface
     {
         return $this->tracerProvider;
     }
 
+    #[\Override]
     public function getTracer(): TracerInterface
     {
         return $this->tracerProvider->getTracer('io.opentelemetry.contrib.php');
@@ -88,6 +97,7 @@ public function instrumentClients($clientsArray): void
         $this->clients = $clientsArray;
     }
 
+    #[\Override]
     public function activate(): bool
     {
         try {

src/Aws/src/Ec2/Detector.php

@@ -63,6 +63,7 @@ public function __construct(ClientInterface $client, RequestFactoryInterface $re
      * if the connection or parsing of the identity document fails.
      *
      */
+    #[\Override]
     public function getResource(): ResourceInfo
     {
         try {

src/Aws/src/Ecs/Detector.php

@@ -92,6 +92,7 @@ public function __construct(
      * - aws.log.stream.names
      * - aws.log.stream.arns
      */
+    #[\Override]
     public function getResource(): ResourceInfo
     {
         $metadataEndpointV4 = getenv(self::ECS_METADATA_KEY_V4);

src/Aws/src/Eks/Detector.php

@@ -59,6 +59,8 @@ public function __construct(
         $this->requestFactory = $requestFactory;
     }
 
+    #[\Override]
+
     public function getResource(): ResourceInfo
     {
         try {

src/Aws/src/Lambda/Detector.php

@@ -38,6 +38,8 @@ class Detector implements ResourceDetectorInterface
     private const AWS_REGION_ENV = 'AWS_REGION';
     private const CLOUD_PROVIDER = 'aws';
 
+    #[\Override]
+
     public function getResource(): ResourceInfo
     {
         $lambdaName = getenv(self::LAMBDA_NAME_ENV);

src/Aws/src/Xray/Propagator.php

@@ -64,6 +64,7 @@ class Propagator implements TextMapPropagatorInterface
     /**
      * Returns list of fields used by HTTPTextFormat
      */
+    #[\Override]
     public function fields(): array
     {
         return [self::AWSXRAY_TRACE_ID_HEADER];
@@ -74,6 +75,7 @@ public function fields(): array
      * X-Amzn-Trace-Id: Root={traceId};Parent={parentId};Sampled={samplingFlag}
      * X-Amzn-Trace-Id: Root=1-5759e988-bd862e3fe1be46a994272793;Parent=53995c3f42cd8ad8;Sampled=1
      */
+    #[\Override]
     public function inject(&$carrier, ?PropagationSetterInterface $setter = null, ?ContextInterface $context = null): void
     {
         $setter = $setter ?? ArrayAccessGetterSetter::getInstance();
@@ -103,6 +105,7 @@ public function inject(&$carrier, ?PropagationSetterInterface $setter = null, ?C
      * This function will parse all parts of the header and validate that it is
      * formatted properly to AWS X-ray standards
      */
+    #[\Override]
     public function extract($carrier, ?PropagationGetterInterface $getter = null, ?ContextInterface $context = null): ContextInterface
     {
         $getter = $getter ?? ArrayAccessGetterSetter::getInstance();

src/Instrumentation/CodeIgniter/src/RequestPropagationGetter.php

@@ -21,13 +21,16 @@ public static function instance(): self
     }
 
     /** @psalm-suppress InvalidReturnType */
+    #[\Override]
     public function keys($carrier): array
     {
         assert($carrier instanceof MessageInterface);
 
         return array_keys($carrier->headers());
     }
 
+    #[\Override]
+
     public function get($carrier, string $key) : ?string
     {
         assert($carrier instanceof MessageInterface);

src/Instrumentation/CodeIgniter/src/ResponsePropagationSetter.php

@@ -30,6 +30,8 @@ public function keys($carrier): array
         return array_keys($carrier->headers());
     }
 
+    #[\Override]
+
     public function set(&$carrier, string $key, string $value): void
     {
         assert($carrier instanceof MessageInterface);

src/Instrumentation/Laravel/src/Propagators/ResponsePropagationSetter.php

@@ -20,6 +20,8 @@ public static function instance(): self
         return $instance ??= new self();
     }
 
+    #[\Override]
+
     public function set(&$carrier, string $key, string $value): void
     {
         assert($carrier instanceof Response);

src/Instrumentation/Slim/src/PsrResponsePropagationSetter.php

@@ -29,6 +29,8 @@ public function keys($carrier): array
         return array_keys($carrier->getHeaders());
     }
 
+    #[\Override]
+
     public function set(&$carrier, string $key, string $value): void
     {
         assert($carrier instanceof ResponseInterface);