rubocop-discourse 3.9.0 → 3.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 416bda6686bc9313c67819f5cdc6e47dd25b68cae808fcb190c1da14b1fda371
-  data.tar.gz: 577720bc7b63aa1fa574ea6da53c9326010e35ce2dc9e367eb888e459d9d17d3
+  metadata.gz: 5b3535299ced6e96016ee33586fe7e9b8e0b910c6cc2a441972acb9e3c6b9207
+  data.tar.gz: 99f0a0dd461aebbc1267dba788730ade31af6a02f0709ae51c8afba65b109d98
 SHA512:
-  metadata.gz: 569fb633072ebff20f3187fd684073d0d1d4ebbaf090cfc173de1c9c794933276ef09beb105d5af2bc639c8b72aa9e221bbcef9d5041d5ab3300dd4a82b64c52
-  data.tar.gz: 6eb8ced2313523c8c6fa0c1681bab7103a9b5b21d97dba683a100bf81bbe082515468d7468c633d46158348d0c92762a32aada56c72afc126b4e28bde5f0b0d8
+  metadata.gz: 48c1e9c95219737c00fd847c9f030f4e3704147b64dd16cc5a79449cec341535d870f498430afc7b47115ad6edb31adddcc54e145e60d100292fef7576e3b689
+  data.tar.gz: 2b19583ee4c90bb79e1dfcfe5919bf1fe9c13786ecd8a8d6b4cfe0bfaa0fc8952ae280a721631d4968b736ac13c34857290698b989bbcf8ce30dafa57bf6c0d7

data/.github/workflows/ci.yml

@@ -34,7 +34,7 @@ jobs:
       - uses: actions/checkout@v3
 
       - name: Release Gem
-        uses: discourse/publish-rubygems-action@v2
+        uses: discourse/publish-rubygems-action@v3
         env:
           RUBYGEMS_API_KEY: ${{ secrets.RUBYGEMS_API_KEY }}
           GIT_EMAIL: team@discourse.org

data/.rspec

@@ -0,0 +1 @@
+--require 'spec_helper'

data/config/default.yml

@@ -1,8 +1,7 @@
 Discourse/NoChdir:
   Enabled: true
   Exclude:
-    - 'spec/**/*' # Specs are run sequentially, so chdir can be used
-    - 'plugins/*/spec/**/*'
+    - "**/spec/**/*" # Specs are run sequentially, so chdir can be used
 
 Discourse/NoTimeNewWithoutArgs:
   Enabled: true
@@ -13,8 +12,8 @@ Discourse/NoURIEscapeEncode:
 Discourse/NoAddReferenceOrAliasesActiveRecordMigration:
   Enabled: true
   Include:
-    - '**/db/migrate/*'
-    - '**/db/post_migrate/*'
+    - "**/db/migrate/*"
+    - "**/db/post_migrate/*"
 
 Discourse/NoNokogiriHtmlFragment:
   Enabled: true
@@ -22,51 +21,45 @@ Discourse/NoNokogiriHtmlFragment:
 Discourse/NoResetColumnInformationInMigrations:
   Enabled: false
   Include:
-    - '**/db/migrate/*'
-    - '**/db/post_migrate/*'
+    - "**/db/migrate/*"
+    - "**/db/post_migrate/*"
 
 # Specs
 
 Discourse/NoDirectMultisiteManipulation:
   Enabled: true
-  Patterns:
-    - _spec.rb
-    - '(?:^|/)spec/'
+  Include:
+    - "**/spec/**/*"
 
 Discourse/TimeEqMatcher:
   Enabled: true
-  Patterns:
-    - _spec.rb
-    - '(?:^|/)spec/'
+  Include:
+    - "**/spec/**/*"
 
 Discourse/NoJsonParseResponse:
   Enabled: false
-  Patterns:
-    - _spec.rb
-    - '(?:^|/)spec/'
+  Include:
+    - "**/spec/**/*"
 
 Discourse/NoMockingJobs:
   Enabled: true
-  Patterns:
-    - _spec.rb
-    - '(?:^|/)spec/'
+  Include:
+    - "**/spec/**/*"
 
 Discourse/OnlyTopLevelMultisiteSpecs:
   Enabled: true
-  Patterns:
-    - _spec.rb
-    - '(?:^|/)spec/'
+  Include:
+    - "**/spec/**/*"
 
 Discourse/NoMixingMultisiteAndStandardSpecs:
   Enabled: true
-  Patterns:
-    - _spec.rb
-    - '(?:^|/)spec/'
+  Include:
+    - "**/spec/**/*"
 
 Discourse/Plugins/CallRequiresPlugin:
   Enabled: true
   Include:
-    - 'app/controllers/**/*'
+    - "app/controllers/**/*"
 
 Discourse/Plugins/UsePluginInstanceOn:
   Enabled: true
@@ -74,16 +67,16 @@ Discourse/Plugins/UsePluginInstanceOn:
 Discourse/Plugins/NamespaceMethods:
   Enabled: true
   Exclude:
-    - '**/spec/**/*'
-    - '**/tasks/**/*.rake'
-    - '**/db/fixtures/**/*'
+    - "**/spec/**/*"
+    - "**/tasks/**/*.rake"
+    - "**/db/fixtures/**/*"
 
 Discourse/Plugins/NamespaceConstants:
   Enabled: true
   Exclude:
-    - '**/spec/**/*'
-    - '**/tasks/**/*.rake'
-    - '**/db/fixtures/**/*'
+    - "**/spec/**/*"
+    - "**/tasks/**/*.rake"
+    - "**/db/fixtures/**/*"
 
 Discourse/Plugins/UseRequireRelative:
   Enabled: true

data/rubocop-core.yml

@@ -39,8 +39,7 @@ Style/GlobalVars:
   Exclude:
     - "lib/tasks/**/*"
     - "script/**/*"
-    - "spec/**/*.rb"
-    - "plugins/*/spec/**/*"
+    - "**/spec/**/*"
 
 Style/InvertibleUnlessCondition:
   Enabled: true
@@ -142,4 +141,4 @@ Lint/SafeNavigationConsistency:
   Enabled: true
 
 Lint/OrAssignmentToConstant:
-  Enabled: true
+  Enabled: true

data/rubocop-discourse.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name = "rubocop-discourse"
-  s.version = "3.9.0"
+  s.version = "3.9.2"
   s.summary = "Custom rubocop cops used by Discourse"
   s.authors = ["Discourse Team"]
   s.license = "MIT"

data/rubocop-rspec.yml

@@ -228,11 +228,11 @@ RSpec/Dialect:
   Enabled: true
   PreferredMethods:
     background: :before
-    scenario:   :it
-    xscenario:  :xit
-    given:      :let
-    given!:     :let!
-    feature:    :describe
+    scenario: :it
+    xscenario: :xit
+    given: :let
+    given!: :let!
+    feature: :describe
 
 RSpecRails/HttpStatus:
   Enabled: true

data/spec/lib/rubocop/cop/discourse/services/empty_lines_around_blocks_spec.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
 RSpec.describe RuboCop::Cop::Discourse::Services::EmptyLinesAroundBlocks,
                :config do
   subject(:cop) { described_class.new(config) }

data/spec/lib/rubocop/cop/discourse/services/group_keywords_spec.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
 RSpec.describe RuboCop::Cop::Discourse::Services::GroupKeywords, :config do
   subject(:cop) { described_class.new(config) }
 

data/spec/lib/rubocop/cop/fabricator_shorthand_spec.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
-describe RuboCop::Cop::Discourse::FabricatorShorthand, :config do
+RSpec.describe RuboCop::Cop::Discourse::FabricatorShorthand, :config do
   subject(:cop) { described_class.new(config) }
 
   let(:config) { RuboCop::Config.new }

data/spec/lib/rubocop/cop/no_add_reference_active_record_migrations_spec.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
-describe RuboCop::Cop::Discourse::NoAddReferenceOrAliasesActiveRecordMigration, :config do
+RSpec.describe RuboCop::Cop::Discourse::NoAddReferenceOrAliasesActiveRecordMigration, :config do
   subject(:cop) { described_class.new(config) }
 
   let(:config) { RuboCop::Config.new }

data/spec/lib/rubocop/cop/no_mixing_multisite_and_standard_specs_spec.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
-describe RuboCop::Cop::Discourse::NoMixingMultisiteAndStandardSpecs, :config do
+RSpec.describe RuboCop::Cop::Discourse::NoMixingMultisiteAndStandardSpecs, :config do
   subject(:cop) { described_class.new(config) }
 
   let(:config) { RuboCop::Config.new }

data/spec/lib/rubocop/cop/no_mocking_jobs_enqueue_spec.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
-describe RuboCop::Cop::Discourse::NoMockingJobs, :config do
+RSpec.describe RuboCop::Cop::Discourse::NoMockingJobs, :config do
   subject(:cop) { described_class.new(config) }
 
   let(:config) { RuboCop::Config.new }

data/spec/lib/rubocop/cop/no_reset_column_information_migrations_spec.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
-describe RuboCop::Cop::Discourse::NoResetColumnInformationInMigrations, :config do
+RSpec.describe RuboCop::Cop::Discourse::NoResetColumnInformationInMigrations, :config do
   subject(:cop) { described_class.new(config) }
 
   let(:config) { RuboCop::Config.new }

data/spec/lib/rubocop/cop/only_top_level_multisite_specs_spec.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
-describe RuboCop::Cop::Discourse::OnlyTopLevelMultisiteSpecs, :config do
+RSpec.describe RuboCop::Cop::Discourse::OnlyTopLevelMultisiteSpecs, :config do
   subject(:cop) { described_class.new(config) }
 
   let(:config) { RuboCop::Config.new }

data/spec/lib/rubocop/cop/plugins/call_requires_plugin_spec.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
 RSpec.describe RuboCop::Cop::Discourse::Plugins::CallRequiresPlugin, :config do
   subject(:cop) { described_class.new(config) }
 

data/spec/lib/rubocop/cop/plugins/namespace_constants_spec.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
 RSpec.describe RuboCop::Cop::Discourse::Plugins::NamespaceConstants, :config do
   subject(:cop) { described_class.new(config) }
 

data/spec/lib/rubocop/cop/plugins/namespace_methods_spec.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
 RSpec.describe RuboCop::Cop::Discourse::Plugins::NamespaceMethods, :config do
   subject(:cop) { described_class.new(config) }
 

data/spec/lib/rubocop/cop/plugins/no_monkey_patching_spec.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
 RSpec.describe RuboCop::Cop::Discourse::Plugins::NoMonkeyPatching, :config do
   subject(:cop) { described_class.new(config) }
 

data/spec/lib/rubocop/cop/plugins/use_plugin_instance_on_spec.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
 RSpec.describe RuboCop::Cop::Discourse::Plugins::UsePluginInstanceOn, :config do
   subject(:cop) { described_class.new(config) }
 

data/spec/lib/rubocop/cop/plugins/use_require_relative_spec.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
 RSpec.describe RuboCop::Cop::Discourse::Plugins::UseRequireRelative, :config do
   subject(:cop) { described_class.new(config) }
 

data/spec/lib/rubocop/cop/time_eq_matcher_spec.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-require "spec_helper"
-
-describe RuboCop::Cop::Discourse::TimeEqMatcher, :config do
+RSpec.describe RuboCop::Cop::Discourse::TimeEqMatcher, :config do
   subject(:cop) { described_class.new(config) }
 
   let(:config) { RuboCop::Config.new }

data/stree-compat.yml

@@ -13,13 +13,13 @@ AllCops:
   SuggestExtensions: false
   DisabledByDefault: true
   Exclude:
-    - 'db/schema.rb'
-    - 'bundle/**/*'
-    - 'vendor/**/*'
-    - '**/node_modules/**/*'
-    - 'public/**/*'
-    - 'plugins/**/gems/**/*'
-    - 'plugins/**/vendor/**/*'
+    - "db/schema.rb"
+    - "bundle/**/*"
+    - "vendor/**/*"
+    - "**/node_modules/**/*"
+    - "public/**/*"
+    - "plugins/**/gems/**/*"
+    - "plugins/**/vendor/**/*"
 
 Discourse:
   Enabled: true
@@ -27,4 +27,4 @@ Discourse:
 Discourse/FabricatorShorthand:
   Enabled: true
   Include:
-    - 'spec/**/*_spec.rb'
+    - "**/spec/**/*"

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-discourse
 version: !ruby/object:Gem::Version
-  version: 3.9.0
+  version: 3.9.2
 platform: ruby
 authors:
 - Discourse Team
-autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-12-02 00:00:00.000000000 Z
+date: 2025-01-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -136,14 +135,13 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 3.12.0
-description: 
-email: 
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
 - ".github/workflows/ci.yml"
 - ".gitignore"
+- ".rspec"
 - ".rubocop.yml"
 - Gemfile
 - LICENSE
@@ -211,7 +209,6 @@ homepage: https://github.com/discourse/rubocop-discourse
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -226,8 +223,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key: 
+rubygems_version: 3.6.2
 specification_version: 4
 summary: Custom rubocop cops used by Discourse
 test_files: []