-
Notifications
You must be signed in to change notification settings - Fork 1
Lint #13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Lint #13
Conversation
There was a problem hiding this 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
publicvisibility modifiers to abstract methods - Reordered union type hints to place
nullat 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.
Description
Explain what this PR does and why it is needed.
Type of change
Checklist
feat: add SQS aspect)composer testpasses locallyRelated issues
Closes #, Relates to #