Add minor cleanup

instrumentation/aws_sdk/README.md

@@ -58,8 +58,6 @@ otel_provider = Aws::Telemetry::OTelProvider.new
 client = Aws::S3::Client.new(telemetry_provider: otel_provider)
 ```
 
-
-
 ## Example
 
 To run the example:

instrumentation/aws_sdk/lib/opentelemetry/instrumentation/aws_sdk/instrumentation.rb

@@ -49,14 +49,17 @@ def require_dependencies
 
         def add_plugins(*targets)
           targets.each do |klass|
-            if telemetry_plugin?
-              klass.add_plugin(AwsSdk::Plugin) unless klass.plugins.include?(Aws::Plugins::Telemetry)
-            else
-              klass.add_plugin(AwsSdk::Plugin)
-            end
+            next if supports_telemetry_plugin?(klass)
+
+            klass.add_plugin(AwsSdk::Plugin)
           end
         end
 
+        def supports_telemetry_plugin?(klass)
+          telemetry_plugin? &&
+            klass.plugins.include?(Aws::Plugins::Telemetry)
+        end
+
         def telemetry_plugin?
           ::Aws.const_defined?('Plugins::Telemetry')
         end

instrumentation/aws_sdk/lib/opentelemetry/instrumentation/aws_sdk/patches/telemetry.rb

@@ -8,7 +8,7 @@ module OpenTelemetry
   module Instrumentation
     module AwsSdk
       module Patches
-        # TODO
+        # Patch for Telemetry Plugin Handler in V3 SDK
         module Handler
           def call(context)
             span_wrapper(context) { @handler.call(context) }