fix rubocop offenses

Author: yiyuan-he

resources/aws/Gemfile

@@ -22,4 +22,4 @@ group :test do
     gem 'base64'
     gem 'mutex_m'
   end
-end
+end

resources/aws/Rakefile

@@ -25,4 +25,4 @@ if RUBY_ENGINE == 'truffleruby'
   task default: %i[test]
 else
   task default: %i[test rubocop yard]
-end
+end

resources/aws/lib/opentelemetry/resource/detector/aws.rb

@@ -15,11 +15,10 @@ module AWS
 
         def detect
           # This will be a composite of all the AWS platform detectors
-          ec2_resource = EC2.detect
+          EC2.detect
 
           # For now, return the EC2 resource directly
           # In the future, we'll implement detection for EC2, ECS, EKS, etc.
-          return ec2_resource
         end
       end
     end

resources/aws/lib/opentelemetry/resource/detector/aws/ec2.rb

@@ -50,7 +50,7 @@ def detect
               resource_attributes[OpenTelemetry::SemanticConventions::Resource::CLOUD_ACCOUNT_ID] = identity['accountId']
               resource_attributes[OpenTelemetry::SemanticConventions::Resource::CLOUD_REGION] = identity['region']
               resource_attributes[OpenTelemetry::SemanticConventions::Resource::CLOUD_AVAILABILITY_ZONE] = identity['availabilityZone']
-              
+
               resource_attributes[OpenTelemetry::SemanticConventions::Resource::HOST_ID] = identity['instanceId']
               resource_attributes[OpenTelemetry::SemanticConventions::Resource::HOST_TYPE] = identity['instanceType']
               resource_attributes[OpenTelemetry::SemanticConventions::Resource::HOST_NAME] = hostname
@@ -65,7 +65,7 @@ def detect
           end
 
           private
-          
+
           # Fetches an IMDSv2 token from the EC2 metadata service
           #
           # @return [String, nil] The token or nil if the request failed

resources/aws/opentelemetry-resource-detector-aws.gemspec

@@ -25,12 +25,4 @@ Gem::Specification.new do |spec|
   spec.required_ruby_version = '>= 2.5.0'
 
   spec.add_dependency 'opentelemetry-sdk', '~> 1.0'
-
-  spec.add_development_dependency 'bundler', '>= 1.17'
-  spec.add_development_dependency 'minitest', '~> 5.0'
-  spec.add_development_dependency 'rake', '~> 12.0'
-  spec.add_development_dependency 'rubocop', '~> 1.42.0'
-  spec.add_development_dependency 'simplecov', '~> 0.17'
-  spec.add_development_dependency 'webmock', '~> 3.7'
-  spec.add_development_dependency 'yard', '~> 0.9'
 end

resources/aws/test/opentelemetry/resource/detector/aws/ec2_test.rb

@@ -12,7 +12,7 @@
   let(:token_path) { '/latest/api/token' }
   let(:identity_document_path) { '/latest/dynamic/instance-identity/document' }
   let(:hostname_path) { '/latest/meta-data/hostname' }
-  
+
   let(:mock_token) { 'mock-token-123456' }
   let(:mock_identity_document) do
     {
@@ -53,12 +53,12 @@
         stub_request(:put, "http://#{ec2_metadata_host}#{token_path}")
           .with(headers: { 'X-aws-ec2-metadata-token-ttl-seconds' => '60' })
           .to_return(status: 200, body: mock_token)
-        
+
         # Stub identity document request
         stub_request(:get, "http://#{ec2_metadata_host}#{identity_document_path}")
           .with(headers: { 'X-aws-ec2-metadata-token' => mock_token })
           .to_return(status: 200, body: mock_identity_document.to_json)
-        
+
         # Stub hostname request
         stub_request(:get, "http://#{ec2_metadata_host}#{hostname_path}")
           .with(headers: { 'X-aws-ec2-metadata-token' => mock_token })
@@ -116,7 +116,7 @@
         stub_request(:put, "http://#{ec2_metadata_host}#{token_path}")
           .with(headers: { 'X-aws-ec2-metadata-token-ttl-seconds' => '60' })
           .to_return(status: 200, body: mock_token)
-        
+
         # Identity document request fails
         stub_request(:get, "http://#{ec2_metadata_host}#{identity_document_path}")
           .with(headers: { 'X-aws-ec2-metadata-token' => mock_token })
@@ -135,7 +135,7 @@
         stub_request(:put, "http://#{ec2_metadata_host}#{token_path}")
           .with(headers: { 'X-aws-ec2-metadata-token-ttl-seconds' => '60' })
           .to_return(status: 200, body: mock_token)
-        
+
         # Identity document is invalid JSON
         stub_request(:get, "http://#{ec2_metadata_host}#{identity_document_path}")
           .with(headers: { 'X-aws-ec2-metadata-token' => mock_token })
@@ -154,12 +154,12 @@
         stub_request(:put, "http://#{ec2_metadata_host}#{token_path}")
           .with(headers: { 'X-aws-ec2-metadata-token-ttl-seconds' => '60' })
           .to_return(status: 200, body: mock_token)
-        
+
         # Successful identity document request
         stub_request(:get, "http://#{ec2_metadata_host}#{identity_document_path}")
           .with(headers: { 'X-aws-ec2-metadata-token' => mock_token })
           .to_return(status: 200, body: mock_identity_document.to_json)
-        
+
         # Hostname request times out
         stub_request(:get, "http://#{ec2_metadata_host}#{hostname_path}")
           .with(headers: { 'X-aws-ec2-metadata-token' => mock_token })
@@ -174,7 +174,7 @@
           'cloud.region' => 'mock-west-2',
           'cloud.availability_zone' => 'mock-west-2a',
           'host.id' => 'i-1234ab56cd7e89f01',
-          'host.type' => 't2.micro-mock',
+          'host.type' => 't2.micro-mock'
           # host.name is missing because the request failed
         }
       end

resources/aws/test/opentelemetry/resource/detector/aws_test.rb

@@ -13,16 +13,17 @@
     before do
       WebMock.disable_net_connect!
       # You'll add stubs for AWS endpoints here
-      stub_request(:put, "http://169.254.169.254/latest/api/token").
-        with(
+      stub_request(:put, 'http://169.254.169.254/latest/api/token')
+        .with(
           headers: {
-            'Accept'=>'*/*',
-            'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
-            'Host'=>'169.254.169.254',
-            'User-Agent'=>'Ruby',
-            'X-Aws-Ec2-Metadata-Token-Ttl-Seconds'=>'60'
-          }).
-          to_return(status: 404, body: "Not Found", headers: {})
+            'Accept' => '*/*',
+            'Accept-Encoding' => 'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
+            'Host' => '169.254.169.254',
+            'User-Agent' => 'Ruby',
+            'X-Aws-Ec2-Metadata-Token-Ttl-Seconds' => '60'
+          }
+        )
+        .to_return(status: 404, body: 'Not Found', headers: {})
     end
 
     after do

resources/aws/test/test_helper.rb

@@ -12,4 +12,4 @@
 require 'minitest/autorun'
 require 'webmock/minitest'
 
-OpenTelemetry.logger = Logger.new($stderr, level: ENV.fetch('OTEL_LOG_LEVEL', 'fatal').to_sym)
+OpenTelemetry.logger = Logger.new($stderr, level: ENV.fetch('OTEL_LOG_LEVEL', 'fatal').to_sym)