Delete some rules specific to the foobara project

foobara · Jan 11, 2024 · 07b4287 · 07b4287
1 parent abae100
commit 07b4287
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 15 deletions.
diff --git a/rules/naming.yml b/rules/naming.yml
@@ -1,20 +1,6 @@
 Naming/AccessorMethodName:
   Enabled: false
 
-Naming/FileName:
-  ExpectMatchingDefinition: true
-  CheckDefinitionPathHierarchy: true
-  Exclude:
-    - spec/lib/**/*
-    - spec/**/*
-    - bin/*
-    - ./require_all_projects.rb
-  CheckDefinitionPathHierarchyRoots:
-    - src
-    - extensions
-    - spec
-  IgnoreExecutableScripts: true
-
 Naming/PredicateName:
   Enabled: false
 

diff --git a/rules/rspec.yml b/rules/rspec.yml
@@ -10,7 +10,7 @@ RSpec/ExampleLength:
 RSpec/ExpectInHook:
   Enabled: false
 
-# Seems like a good rule but for now going to use this repository as a bit of a monorepo so disabling for now.
+# Deprecated and wonky
 RSpec/FilePath:
   Enabled: false
 
@@ -42,6 +42,8 @@ RSpec/NoExpectationExample:
 RSpec/NotToNot:
   EnforcedStyle: to_not
 
+# Would be really nice to turn this on if we can exclude namespaces like Foobara so we don´t have to have foobara/
+# prefixing every path.
 RSpec/SpecFilePathFormat:
   Enabled: false