diff --git a/gems/smithy/lib/smithy/forge/client.rb b/gems/smithy/lib/smithy/forge/client.rb
index 3742deb1..6ae73f38 100644
--- a/gems/smithy/lib/smithy/forge/client.rb
+++ b/gems/smithy/lib/smithy/forge/client.rb
@@ -24,6 +24,7 @@ def forge
 
       private
 
+      # rubocop:disable Metrics/AbcSize
       def source_files
         Enumerator.new do |e|
           e.yield "#{@gem_name}.gemspec", render_gemspec
diff --git a/gems/smithy/spec/interfaces/client/shapes_spec.rb b/gems/smithy/spec/interfaces/client/shapes_spec.rb
index b8164fe6..97959dcd 100644
--- a/gems/smithy/spec/interfaces/client/shapes_spec.rb
+++ b/gems/smithy/spec/interfaces/client/shapes_spec.rb
@@ -37,7 +37,9 @@
         end
 
         it 'has a type variation of the shape when applicable' do
-          skip("Test does not expect the generated #{id} to have a type") unless %w[structure union].include?(shape['type'])
+          unless %w[structure union].include?(shape['type'])
+            skip("Test does not expect the generated #{id} to have a type")
+          end
 
           expected_type = Object.const_get("#{types_module}::#{shape_name}")
           expect(generated_shape.type).to eq(expected_type)
@@ -89,7 +91,9 @@
 
         context 'key and value members' do
           it 'are shapes of expected member names, shapes and contains traits when applicable' do
-            skip("Test does not expect the generated #{id} to have a key/value members") if shape['key'].nil? && shape['value'].nil?
+            if shape['key'].nil? && shape['value'].nil?
+              skip("Test does not expect the generated #{id} to have a key/value members")
+            end
 
             expect(generated_shape.key.name).to eq('key')
             expect(generated_shape.value.name).to eq('value')