Skip to content

Commit e9cd834

Browse files
authored
Merge pull request #57 from rspec/add-rubocop
Add rubocop
2 parents ac34412 + 91f2caa commit e9cd834

File tree

6 files changed

+407
-4
lines changed

6 files changed

+407
-4
lines changed

.github/workflows/ci.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,18 @@ concurrency:
1616
cancel-in-progress: true
1717

1818
jobs:
19+
rubocop:
20+
name: Rubocop
21+
runs-on: 'ubuntu-20.04'
22+
steps:
23+
- uses: actions/checkout@v4
24+
- uses: ruby/setup-ruby@v1
25+
with:
26+
ruby-version: '3.2'
27+
- run: script/update_rubygems_and_install_bundler
28+
- run: bundle install --standalone
29+
- run: bundle exec rubocop -c .rubocop.yml
30+
1931
test:
2032
name: 'Ruby: ${{ matrix.ruby }}, Rails: ${{ matrix.env.RAILS_VERSION }}'
2133
runs-on: ubuntu-20.04

.rubocop.yml

Lines changed: 64 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,64 @@
1+
inherit_from:
2+
- .rubocop_rspec_base.yml
3+
4+
AllCops:
5+
NewCops: enable
6+
TargetRubyVersion: 2.0
7+
SuggestExtensions: false
8+
Exclude:
9+
- bin/**/*
10+
- tmp/**/*
11+
- bundle/**/*
12+
13+
# TODO: Remove
14+
Gemspec/DevelopmentDependencies:
15+
Enabled: false
16+
17+
Gemspec/DeprecatedAttributeAssignment:
18+
Enabled: false
19+
20+
Gemspec/RequiredRubyVersion:
21+
Enabled: false
22+
23+
# TODO: Remove
24+
Gemspec/RubyVersionGlobalsUsage:
25+
Enabled: false
26+
27+
Layout/LineLength:
28+
Max: 120
29+
30+
# Over time we'd like to get this down, but this is what we're at now.
31+
Metrics/AbcSize:
32+
Max: 28
33+
34+
# Over time we'd like to get this down, but this is what we're at now.
35+
Metrics/BlockLength:
36+
Max: 86
37+
Exclude:
38+
- spec/**/*
39+
40+
Metrics/ModuleLength:
41+
Max: 140
42+
43+
# Over time we'd like to get this down, but this is what we're at now.
44+
Metrics/PerceivedComplexity:
45+
Max: 10
46+
47+
Naming/FileName:
48+
Exclude:
49+
- lib/rspec-activemodel-mocks.rb
50+
51+
Style/ClassAndModuleChildren:
52+
Exclude:
53+
- lib/rspec/active_model/mocks/mocks.rb
54+
55+
# Still support older Rubies
56+
Style/ExpandPathArguments:
57+
Enabled: false
58+
59+
Style/NumericLiterals:
60+
Exclude:
61+
- templates/sample/db/*
62+
63+
Style/OptionalBooleanParameter:
64+
Enabled: false

0 commit comments

Comments
 (0)