Skip to content

Conversation

@lucas-angeli-gimenes
Copy link
Collaborator

Description

Explain what this PR does and why it is needed.

Type of change

  • Bug fix
  • New feature
  • Documentation
  • Performance/Refactor
  • CI/CD/Chore

Checklist

  • Title follows Conventional Commits (e.g., feat: add SQS aspect)
  • Tests added/updated
  • composer test passes locally
  • Documentation updated (README/Docs)
  • No breaking changes (or documented if any)

Related issues

Closes #, Relates to #

Copilot AI review requested due to automatic review settings February 4, 2026 13:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR applies automated code formatting fixes using php-cs-fixer to ensure consistent code style across the codebase.

Changes:

  • Alphabetically sorted import statements according to PSR standards
  • Removed unused imports and added missing imports for global classes
  • Fixed spacing around operators (not operator, concatenation)
  • Added explicit public visibility modifiers to abstract methods
  • Reordered union type hints to place null at the end

Reviewed changes

Copilot reviewed 53 out of 53 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/bootstrap.php Added imports for ClassLoader and Runtime, fixed spacing in ! defined()
tests/Unit/SwitcherTest.php Reordered imports alphabetically
tests/Unit/Support/UriTest.php Reordered imports alphabetically
tests/Unit/Support/SpanScopeTest.php Moved SpanScope import to alphabetical position
tests/Unit/Support/HyperfGuzzleTest.php Removed unused imports (ClientFactory, ReflectionAttribute)
tests/Unit/Propagator/HeadersPropagatorTest.php Reordered imports alphabetically
tests/Unit/Middleware/TraceMiddlewareTest.php Moved Hyperf imports before OpenTelemetry imports (alphabetical)
tests/Unit/Middleware/MetricMiddlewareTest.php Moved Hyperf imports before OpenTelemetry imports
tests/Unit/Middleware/AbstractMiddlewareTest.php Reordered imports alphabetically
tests/Unit/Listener/TraceFlushListenerTest.php Removed unused MetricFlushListener import
tests/Unit/Listener/OtelShutdownListenerTest.php Added RuntimeException import, added missing docblock
tests/Unit/Listener/DbQueryExecutedListenerTest.php Moved Hyperf imports before OpenTelemetry imports
tests/Unit/InstrumentationTest.php Moved Instrumentation import to alphabetical position
tests/Unit/Factory/Trace/TracerProviderFactoryTest.php Moved Hyperf imports before OpenTelemetry imports
tests/Unit/Factory/Trace/Sampler/AlwaysOnSamplerFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/Trace/Processor/SimpleSpanProcessorFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/Trace/Processor/BatchSpanProcessorFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/Trace/Exporter/StdoutTraceExporterFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/Trace/Exporter/OtlpHttpTraceExporterFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/SDKBuilderTest.php Moved SDKBuilder import to alphabetical position
tests/Unit/Factory/OTelResourceFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/Metric/MeterProviderFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/Metric/Exporter/StdoutMetricExporterFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/Metric/Exporter/OtlpHttpMetricExporterFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/Log/Processor/SimpleLogProcessorFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/Log/Processor/BatchLogProcessorFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/Log/LoggerProviderFactoryTest.php Moved Hyperf imports before OpenTelemetry imports
tests/Unit/Factory/Log/Exporter/StdoutLogExporterFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/Log/Exporter/OtlpHttpLogExporterFactoryTest.php Moved factory import to alphabetical position
tests/Unit/Factory/CachedInstrumentationFactoryTest.php Moved Hyperf imports before OpenTelemetry imports
tests/Unit/ConfigProviderTest.php Moved Hyperf imports before OpenTelemetry imports
tests/Unit/Aspect/RedisAspectTest.php Moved Hyperf imports before OpenTelemetry imports
tests/Unit/Aspect/MongoAspectTest.php Added imports for Collection, ReflectionClass, ReflectionProperty; fixed spacing
tests/Unit/Aspect/GuzzleClientAspectTest.php Moved Hyperf imports before OpenTelemetry imports, added blank line
tests/Unit/Aspect/Aws/SnsClientAspectTest.php Moved Hyperf imports before OpenTelemetry imports
tests/Unit/Aspect/Aws/DynamoDbClientAspectTest.php Moved Hyperf imports before OpenTelemetry imports
src/Support/SpanScope.php Reordered union type to place null at end
src/Middleware/TraceMiddleware.php Removed unused imports, reordered remaining imports
src/Middleware/MetricMiddleware.php Moved Uri import to alphabetical position
src/Middleware/AbstractMiddleware.php Reordered imports alphabetically
src/Listener/TraceFlushListener.php Added public visibility to abstract method implementations
src/Listener/MetricFlushListener.php Added public visibility to abstract method implementations
src/Listener/DbQueryExecutedListener.php Moved AbstractInstrumenter import to alphabetical position
src/Listener/AbstractFlushListener.php Added public visibility to abstract methods, fixed spacing
src/Instrumentation.php Moved SpanScope import to alphabetical position
src/Factory/Trace/TracerProviderFactory.php Moved Hyperf imports before OpenTelemetry imports
src/Factory/Log/LoggerProviderFactory.php Moved Hyperf imports before OpenTelemetry imports
src/ConfigProvider.php Reordered imports, removed duplicate OTelResourceFactory import
src/Aspect/MongoAspect.php Removed unnecessary AbstractAspect import (same namespace)
src/Aspect/GuzzleClientAspect.php Moved Hyperf imports before OpenTelemetry imports, fixed concatenation spacing
src/Aspect/Aws/SnsClientAspect.php Moved AbstractAspect import to alphabetical position
src/Aspect/Aws/DynamoDbClientAspect.php Moved AbstractAspect import to alphabetical position
publish/open-telemetry.php Moved Hyperf imports before OpenTelemetry imports

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lucas-angeli-gimenes lucas-angeli-gimenes merged commit 278852a into main Feb 4, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants