From 3b0e126e389236b58c83d3d742a81e0c1258e840 Mon Sep 17 00:00:00 2001
From: Sebastian Serth <Sebastian.Serth@hpi.de>
Date: Wed, 18 Jan 2023 10:48:53 +0100
Subject: [PATCH] Remove outdated Rubocop config

---
 .rubocop.yml         |  1 +
 .rubocop/layout.yml  |  1 -
 .rubocop/lint.yml    | 11 -----------
 .rubocop/metrics.yml |  2 --
 .rubocop/style.yml   | 29 -----------------------------
 .rubocop_todo.yml    |  7 -------
 6 files changed, 1 insertion(+), 50 deletions(-)

diff --git a/.rubocop.yml b/.rubocop.yml
index 6f1f724d..707f2447 100644
--- a/.rubocop.yml
+++ b/.rubocop.yml
@@ -1,4 +1,5 @@
 require:
+  - rubocop-capybara
   - rubocop-performance
   - rubocop-rails
   - rubocop-rspec
diff --git a/.rubocop/layout.yml b/.rubocop/layout.yml
index 5ada29a1..0fc42d0a 100644
--- a/.rubocop/layout.yml
+++ b/.rubocop/layout.yml
@@ -24,7 +24,6 @@ Layout/HashAlignment:
 
 Layout/LineLength:
   Exclude:
-    - "api/**/*"
     - "config/**/*"
     - "db/**/*"
     - "spec/**/*"
diff --git a/.rubocop/lint.yml b/.rubocop/lint.yml
index 98fdac2d..db1306bb 100644
--- a/.rubocop/lint.yml
+++ b/.rubocop/lint.yml
@@ -7,14 +7,3 @@
 Lint/AmbiguousBlockAssociation:
   Exclude:
     - "spec/**/*_spec.rb"
-
-#Lint/ShadowingOuterLocalVariable:
-#  Exclude:
-#    - "spec/**/*_spec.rb"
-#
-##
-## Assume we know what we do when resucing exception, as we do it much less often
-## than we actually should.
-##
-#Lint/SuppressedException:
-#  Enabled: false
diff --git a/.rubocop/metrics.yml b/.rubocop/metrics.yml
index bfb4e313..eefe6d80 100644
--- a/.rubocop/metrics.yml
+++ b/.rubocop/metrics.yml
@@ -14,9 +14,7 @@ Metrics/BlockLength:
     - "spec/**/*"
     - "config/**/*"
     - "**/*.rake"
-    - "api/**/*"
     - "Rakefile"
-    - "Guardfile"
     - "**/*/Rakefile"
   Max: 50
 
diff --git a/.rubocop/style.yml b/.rubocop/style.yml
index 1156b04c..8935a51a 100644
--- a/.rubocop/style.yml
+++ b/.rubocop/style.yml
@@ -52,14 +52,6 @@ Style/GuardClause:
 Style/IfUnlessModifier:
   Enabled: false
 
-##
-## Scripts might include on top-level
-##
-#Style/MixinUsage:
-#  Exclude:
-#    - "scripts/**/*"
-#    - "services/*/scripts/**/*"
-
 #
 # Well, we do this. To often to disable them. Studid.
 #
@@ -72,27 +64,6 @@ Style/MultilineBlockChain:
 Style/RaiseArgs:
   EnforcedStyle: compact
 
-##
-## Quick single line rescues in specs
-##
-#Style/RescueModifier:
-#  Exclude:
-#    - "spec/**/*_spec.rb"
-#
-##
-## Quick single line rescues in specs
-##
-#Style/RescueStandardError:
-#  Enabled: false
-
-##
-## Often used pattern in chaining subjects in specs
-##
-#Style/Semicolon:
-#  Exclude:
-#    - "services/*/spec/**/*_spec.rb"
-#    - "spec/**/*_spec.rb"
-
 Style/SignalException:
   EnforcedStyle: only_raise
 
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index 6477994c..f7ea2557 100644
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -1,12 +1,5 @@
 # TODOs
 
-# The API seems to use huge amounts of {...} blocks for it's API design.
-#
-Style/BlockDelimiters:
-  Exclude:
-    - api/**/*.rb
-
-
 # Lots of line are too long too.
 #
 Layout/LineLength: