rubocop-codetakt 0.8.3 → 0.82.0.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +5 -5
- data/.gitignore +10 -0
- data/.rspec +1 -0
- data/.rubocop.yml +12 -0
- data/.travis.yml +4 -0
- data/CHANGELOG.md +189 -43
- data/Gemfile +10 -0
- data/{LICENSE → LICENSE.txt} +3 -1
- data/README.md +36 -19
- data/Rakefile +4 -0
- data/bin/console +14 -0
- data/bin/setup +8 -0
- data/config/performance.yml +6 -0
- data/config/rails.yml +53 -3
- data/config/rspec.yml +64 -11
- data/config/rubocop.yml +351 -15
- data/exe/rubocop-codetakt +10 -0
- data/lib/rubocop/codetakt.rb +4 -0
- data/lib/rubocop/codetakt/cli.rb +46 -0
- data/lib/rubocop/codetakt/version.rb +5 -0
- data/lib/rubocop_codetakt.rb +2 -0
- data/rubocop-codetakt.gemspec +25 -0
- data/templates/.rubocop.yml +12 -0
- metadata +35 -44
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
|
-
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
2
|
+
SHA256:
|
|
3
|
+
metadata.gz: 9ce6c7a74ef3e10599646185d4cdb5f24e9b7c28a642dab9fcd991320eb0a266
|
|
4
|
+
data.tar.gz: 9c5ba3dc1a0cb4ee4a3573fc3d94ad0f18dcbf5b0920690a6335cb57604e3bd2
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: 758dda084a7a4ee3463c853642c706fea238a7375c936b5ca36cc2a2a3661df1d87bb37442dae25509c13558cc4844fc8a2d45f62489b6f2818f7958041cfe9a
|
|
7
|
+
data.tar.gz: c1941541ff7a5b8a3f605e3de6732ddad53f55e6a171e548b4da5ad809afb478f456370ac458ba765e4d249314473e8ff82bd180ba7e66f75b3364a6532c6ac9
|
data/.gitignore
ADDED
data/.rspec
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
--require spec_helper
|
data/.rubocop.yml
ADDED
|
@@ -0,0 +1,12 @@
|
|
|
1
|
+
inherit_gem:
|
|
2
|
+
rubocop-codetakt:
|
|
3
|
+
- "config/rubocop.yml"
|
|
4
|
+
# uncomment if use performance cops
|
|
5
|
+
- "config/performance.yml"
|
|
6
|
+
# uncomment if use rails cops
|
|
7
|
+
- "config/rails.yml"
|
|
8
|
+
# uncomment if use rspec cops
|
|
9
|
+
- "config/rspec.yml"
|
|
10
|
+
|
|
11
|
+
AllCops:
|
|
12
|
+
TargetRubyVersion: 2.5
|
data/.travis.yml
ADDED
data/CHANGELOG.md
CHANGED
|
@@ -1,73 +1,219 @@
|
|
|
1
|
-
# rubocop-codetakt
|
|
1
|
+
# rubocop-codetakt:
|
|
2
2
|
|
|
3
|
-
## v0.
|
|
3
|
+
## v0.82.0.0 (2020-04-20)
|
|
4
4
|
|
|
5
|
-
|
|
5
|
+
* Fork from onk/onkcop to codetakt/rubocop-codetakt
|
|
6
|
+
* Follow upstream renamings
|
|
7
|
+
* Separate the file of performance settings
|
|
8
|
+
* Clarify requiring the `rubocop-rails` gem in the Rails settings file
|
|
9
|
+
* Refine dependency settings
|
|
10
|
+
* Remove `Layout/DotPosition` cop'
|
|
11
|
+
* Remove `Layout/IndentationConsistency` cop
|
|
12
|
+
* Remove `Style/BracesAroundHashParameters` cop
|
|
13
|
+
* Remove `Style/MethodCalledOnDoEndBlock`
|
|
14
|
+
* Remove `Style/StringLiteralsInInterpolation` cop
|
|
15
|
+
* Remove `Style/StringLiterals` cop
|
|
16
|
+
* Change `Metrics/AbcSize` cop option
|
|
17
|
+
* Change `Metrics/PerceivedComplexity` cop option
|
|
18
|
+
* Change `Layout/MultilineMethodCallIndentation` cop option
|
|
19
|
+
* Change `Style/TrailingCommaInArguments` cop option
|
|
20
|
+
* Change `Style/TrailingCommaInArrayLiteral` cop option
|
|
21
|
+
* Change `Style/TrailingCommaInHashLiteral` cop option
|
|
22
|
+
* Change `Rails/ApplicationRecord` cop option
|
|
6
23
|
|
|
7
|
-
|
|
8
|
-
* Backport "Disable RSpec/EmptyLineAfterFinalLet cop"
|
|
9
|
-
* From: https://github.com/onk/onkcop/pull/30
|
|
24
|
+
## v0.53.0.3 (2019-01-20)
|
|
10
25
|
|
|
11
|
-
|
|
26
|
+
* Ignored `Style/YodaCondition` (@sue445)
|
|
12
27
|
|
|
13
|
-
|
|
28
|
+
## v0.53.0.2 (2018-10-11)
|
|
14
29
|
|
|
15
|
-
*
|
|
30
|
+
* Remove `Style/EmptyLineAfterGuardClause` cop's configuration
|
|
16
31
|
|
|
17
|
-
## v0.
|
|
32
|
+
## v0.53.0.1 (2018-08-17)
|
|
18
33
|
|
|
19
|
-
|
|
34
|
+
* Relax rubocop dependency for `rubocop-rspec` (@sue445)
|
|
20
35
|
|
|
21
|
-
|
|
22
|
-
* Disable the RSpec/DescribedClass cop for the request specs.
|
|
36
|
+
## v0.53.0.0 (2018-03-09)
|
|
23
37
|
|
|
24
|
-
|
|
38
|
+
* Update `rubocop` v0.53.0 and `rubocop-rspec` v1.24.0
|
|
39
|
+
* `Style/TrailingCommaInLiteral` cop is separated to `Style/TrailingCommaInArrayLiteral` and `Style/TrailingCommaInHashLiteral`
|
|
40
|
+
* `Performance/HashEachMethods` cop is removed
|
|
41
|
+
* Disable new `Naming/UncommunicativeMethodParamName` cop
|
|
42
|
+
* Enable `Style/FormatStringToken` cop
|
|
43
|
+
* Enable new `Style/EmptyLineAfterGuardClause` cop
|
|
44
|
+
* Enable `Lint/Void` cop's mutating methods check
|
|
45
|
+
* Remove `TargetRailsVersion` from template
|
|
46
|
+
* Change `Layout/SpaceInsideBlockBraces` cop option
|
|
25
47
|
|
|
26
|
-
|
|
48
|
+
## v0.52.1.1 (2018-01-11)
|
|
27
49
|
|
|
28
|
-
*
|
|
50
|
+
* Update `rubocop-rspec` to v1.22.0
|
|
51
|
+
* Change new `RSpec/ExpectChange` cop to `block` style
|
|
29
52
|
|
|
30
|
-
## v0.3.0 (2017-04-03)
|
|
31
53
|
|
|
32
|
-
|
|
54
|
+
## v0.52.1.0 (2017-12-28)
|
|
33
55
|
|
|
34
|
-
*
|
|
35
|
-
*
|
|
56
|
+
* Update `rubocop` to v0.52.1
|
|
57
|
+
* Use `Layout/SpaceBeforeBlockBraces` cop's default configuration
|
|
58
|
+
* Update `TargetRubyVersion` of auto-generate configuration to 2.5
|
|
59
|
+
* `Lint/AmbiguousBlockAssociation`, `Style/BlockDelimiters` and
|
|
60
|
+
`Style/Semicolon` cops are now exclude only `spec/**/*_spec.rb` instead of `spec/**/*`
|
|
36
61
|
|
|
37
|
-
## v0.2.3 (2017-03-27)
|
|
38
62
|
|
|
39
|
-
|
|
63
|
+
## v0.52.0.0 (2017-12-20)
|
|
40
64
|
|
|
41
|
-
*
|
|
42
|
-
|
|
65
|
+
* Update `rubocop` v0.52.0 and `rubocop-rspec` v1.21.0
|
|
66
|
+
* Remove `DisplayCopNames` configuration
|
|
67
|
+
* Enable new `Style/ClassStructure` cop
|
|
68
|
+
* Disalbe `Style/FormatStringToken` cop
|
|
69
|
+
* Change `Layout/SpaceBeforeBlockBraces` cop's empty braces style to `space`
|
|
70
|
+
* Change `Style/RescueStandardError` cop to implicit style
|
|
71
|
+
* Change `RSpec/MultipleExpectations` cop to `AggregateFailuresByDefault` style
|
|
72
|
+
* Exclude `bin/setup`, `bin/update` from `Style/MixinUsage` cop
|
|
43
73
|
|
|
44
|
-
## v0.2.2 (2017-03-27)
|
|
45
74
|
|
|
46
|
-
|
|
75
|
+
## v0.51.0.1 (2017-11-11)
|
|
47
76
|
|
|
48
|
-
*
|
|
77
|
+
* Update `rubocop-rspec` to v1.20.0
|
|
78
|
+
* Disable new `RSpec/ContextWording` cop
|
|
79
|
+
* Disable `Rails/Delegate`, `Rails/Exit`, `Rails/FilePath`,
|
|
80
|
+
`Rails/PluralizationGrammar` and `Rails/Present` cops
|
|
81
|
+
* Enable `Rails/NotNullColumn` cop
|
|
49
82
|
|
|
50
|
-
See the upstream's release note: <https://github.com/bbatsov/rubocop/releases/tag/v0.48.0>
|
|
51
83
|
|
|
52
|
-
## v0.
|
|
84
|
+
## v0.51.0.0 (2017-10-20)
|
|
53
85
|
|
|
54
|
-
|
|
86
|
+
* Update `rubocop` v0.51.0 and `rubocop-rspec` v1.19.0
|
|
87
|
+
* Enable `Lint/ReturnInVoidContext` cop
|
|
88
|
+
* Add `staging` env to `Rails/UnknownEnv` cop
|
|
89
|
+
* Add comments to `Style/SafeNavigation` and `Lint/UnusedMethodArgument` cop
|
|
55
90
|
|
|
56
|
-
* Fix README's typo.
|
|
57
91
|
|
|
58
|
-
## v0.
|
|
92
|
+
## v0.50.0.0 (2017-10-12)
|
|
59
93
|
|
|
60
|
-
|
|
94
|
+
* Update `rubocop` v0.50.0 and `rubocop-rspec` v1.18.0
|
|
95
|
+
* Disable `Performance/HashEachMethods` cop
|
|
96
|
+
* Disable new `Style/OrAssignment` cop
|
|
97
|
+
* Change cop's department from `Style` to `Naming`
|
|
98
|
+
* Disable new `Lint/RescueWithoutErrorClass` cop
|
|
99
|
+
* `node_modules` directory is now excluded by rubocop as default
|
|
100
|
+
* Enable `Style/YodaCondition` cop with `equality_operators_only` style
|
|
61
101
|
|
|
62
|
-
* Enabled:
|
|
63
|
-
* `DisplayCopNames`
|
|
64
102
|
|
|
65
|
-
|
|
66
|
-
* `Style/DoubleNegation`
|
|
103
|
+
## v0.49.1.1 (2017-09-11)
|
|
67
104
|
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
|
|
73
|
-
|
|
105
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.49.1.0...v0.49.1.1)
|
|
106
|
+
|
|
107
|
+
* Update `rubocop-rspec` v1.16.0
|
|
108
|
+
* Disable new `RSpec/ReturnFromStub` cop.
|
|
109
|
+
* Exclude `config/environments/*.rb` from Metrics/BlockLength cop.
|
|
110
|
+
* Disable `Performance/Casecmp` cop.
|
|
111
|
+
* Disable `RSpec/EmptyLineAfterFinalLet` cop.
|
|
112
|
+
* Relax `RSpec/NestedGroups` cop.
|
|
113
|
+
|
|
114
|
+
|
|
115
|
+
## v0.49.1.0 (2017-05-31)
|
|
116
|
+
|
|
117
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.49.0.0...v0.49.1.0)
|
|
118
|
+
|
|
119
|
+
* Update `rubocop` v0.49.1.
|
|
120
|
+
* Change `Style/TrailingCommaInArguments` to `comma` style.
|
|
121
|
+
|
|
122
|
+
|
|
123
|
+
## v0.49.0.0 (2017-05-25)
|
|
124
|
+
|
|
125
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.48.1.1...v0.49.0.0)
|
|
126
|
+
|
|
127
|
+
* Update `rubocop` v0.49.0 and `rubocop-rspec` v1.15.1
|
|
128
|
+
* Enable `Lint/AmbiguousBlockAssociation` cop except `spec/**/*`
|
|
129
|
+
* Change some cops to `Layout` department
|
|
130
|
+
* Disable new `Style/YodaCondition` cop
|
|
131
|
+
|
|
132
|
+
|
|
133
|
+
## v0.48.1.1 (2017-04-18)
|
|
134
|
+
|
|
135
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.48.1.0...v0.48.1.1)
|
|
136
|
+
|
|
137
|
+
* Disable `Lint/AmbiguousBlockAssociation` cop.
|
|
138
|
+
|
|
139
|
+
|
|
140
|
+
## v0.48.1.0 (2017-04-17)
|
|
141
|
+
|
|
142
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.48.0.0...v0.48.1.0)
|
|
143
|
+
|
|
144
|
+
* Update `rubocop` v0.48.1.
|
|
145
|
+
* Enable `Rails/Blank` cop.
|
|
146
|
+
* Enable `Lint/AmbiguousBlockAssociation` cop.
|
|
147
|
+
* Remove Exclude option from `Style/MixinGrouping` cop.
|
|
148
|
+
* Change `Style/NumericLiterals` cop to `Strict`.
|
|
149
|
+
|
|
150
|
+
|
|
151
|
+
## v0.48.0.0 (2017-03-29)
|
|
152
|
+
|
|
153
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.47.1.2...v0.48.0.0)
|
|
154
|
+
|
|
155
|
+
* Update `rubocop` v0.48.0.
|
|
156
|
+
* Disable new `Lint/AmbiguousBlockAssociation` cop.
|
|
157
|
+
* Disable new `Rails/Blank` cop.
|
|
158
|
+
* Disable `Style/SymbolArray`, `StyleWordArray` cop.
|
|
159
|
+
* Change `Style/MultilineMethodCallIndentation` to `indented_relative_to_receiver` style.
|
|
160
|
+
* Exclude RSpec directory from new `Style/MixinGrouping` cop.
|
|
161
|
+
* Ignore `node_modules` dir.
|
|
162
|
+
* Add `TargetRailsVersion` to README and template
|
|
163
|
+
|
|
164
|
+
|
|
165
|
+
## v0.47.1.2 (2017-03-01)
|
|
166
|
+
|
|
167
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.47.1.1...v0.47.1.2)
|
|
168
|
+
|
|
169
|
+
* Change `Style/StringLiteralsInInterpolation` to `double_quotes` style.
|
|
170
|
+
|
|
171
|
+
|
|
172
|
+
## v0.47.1.1 (2017-03-01)
|
|
173
|
+
|
|
174
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.47.1.0...v0.47.1.1)
|
|
175
|
+
|
|
176
|
+
* Update `rubocop-rspec` v1.12.0.
|
|
177
|
+
* Enable `Style/StringMethods` cop.
|
|
178
|
+
* Enable `Style/StringLiteralsInInterpolation` cop.
|
|
179
|
+
* Change `RSpec/DescribedClass` to `explicit` style.
|
|
180
|
+
* Exclude `*.gemspec` from `Metrics/BlockLength` cop.
|
|
181
|
+
* Disable `Lint/EmptyWhen` cop.
|
|
182
|
+
|
|
183
|
+
|
|
184
|
+
## v0.47.1.0 (2017-01-20)
|
|
185
|
+
|
|
186
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.46.0.1...v0.47.1.0)
|
|
187
|
+
|
|
188
|
+
* Update to `rubocop` v0.47.1 and `rubocop-rspec` v1.10.0.
|
|
189
|
+
* Remove `Style/SingleLineBlockParams` cop config because disabled by default.
|
|
190
|
+
* Disable new `Security/YAMLLoad` cop.
|
|
191
|
+
|
|
192
|
+
|
|
193
|
+
## v0.46.0.1 (2017-01-07)
|
|
194
|
+
|
|
195
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.46.0.0...v0.46.0.1)
|
|
196
|
+
|
|
197
|
+
* Update to `rubocop-rspec` v1.9.1.
|
|
198
|
+
* Disable `RSpec/MessageExpectation` cop that is replaced with a new cop: `RSpec/MessageSpies`.
|
|
199
|
+
* Add CLI for setup `.rubocop.yml`.
|
|
200
|
+
|
|
201
|
+
|
|
202
|
+
## v0.46.0.0 (2016-11-30)
|
|
203
|
+
|
|
204
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.45.0.0...v0.46.0.0)
|
|
205
|
+
|
|
206
|
+
* Update to `rubocop` v0.46.0.
|
|
207
|
+
* Use new `Style/EmptyMethod` cop with `expanded` style.
|
|
208
|
+
* Use `Style/TernaryParentheses` cop `require_parentheses_when_complex` style.
|
|
209
|
+
|
|
210
|
+
|
|
211
|
+
## v0.45.0.0 (2016-11-02)
|
|
212
|
+
[full changelog](https://github.com/onk/onkcop/compare/v0.44.1.1...v0.45.0.0)
|
|
213
|
+
|
|
214
|
+
* Update to `rubocop` v0.45.0 and `rubocop-rspec` v1.8.0.
|
|
215
|
+
* Disable new `RSpec/ImplicitExpect` cop.
|
|
216
|
+
* Explicitly enable `Rspec/MessageExpectation` cop that is now disabled by default.
|
|
217
|
+
* Exclude Gemfile, Guardfile on `Metrics/BlockLength`.
|
|
218
|
+
* Disable `Style/TernaryParentheses` cop.
|
|
219
|
+
* Enable `Rails/HttpPositionalArguments` cop that fixes bug.
|
data/Gemfile
ADDED
|
@@ -0,0 +1,10 @@
|
|
|
1
|
+
source "https://rubygems.org"
|
|
2
|
+
|
|
3
|
+
# Specify your gem's dependencies in rubocop-codetakt.gemspec
|
|
4
|
+
gemspec
|
|
5
|
+
|
|
6
|
+
gem "rake", "~> 12.0"
|
|
7
|
+
gem "rspec", "~> 3.0"
|
|
8
|
+
gem "rubocop-performance", "~> 1.5.0"
|
|
9
|
+
gem "rubocop-rails", "~> 2.5.0"
|
|
10
|
+
gem "rubocop-rspec", "~> 1.38.0"
|
data/{LICENSE → LICENSE.txt}
RENAMED
|
@@ -1,6 +1,8 @@
|
|
|
1
1
|
The MIT License (MIT)
|
|
2
2
|
|
|
3
|
-
Copyright
|
|
3
|
+
Copyright for portions of project rubocop-codetakt are held by Takafumi
|
|
4
|
+
ONAKA, 2019 as part of project onkcop. All other copyright for project
|
|
5
|
+
rubocop-codetakt are held by codeTakt, 2020.
|
|
4
6
|
|
|
5
7
|
Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
6
8
|
of this software and associated documentation files (the "Software"), to deal
|
data/README.md
CHANGED
|
@@ -1,38 +1,55 @@
|
|
|
1
|
-
#
|
|
1
|
+
# rubocop-codetakt
|
|
2
2
|
|
|
3
|
-
|
|
4
|
-
|
|
3
|
+
[](https://badge.fury.io/rb/rubocop-codetakt)
|
|
4
|
+
|
|
5
|
+
This includes the RuboCop configuration used by codeTakt. It is for the unification and linfting of coding styles.
|
|
6
|
+
|
|
7
|
+
**IMPORTANT** This repository is forked from onk/onkcop v0.53.0.3 (979b8e74eda394434e49caaee31d3f3d781d14f6). And it has been modified for our team. Therefore, it should not be regarded as a successor.
|
|
5
8
|
|
|
6
9
|
## Usage
|
|
7
10
|
|
|
8
|
-
|
|
11
|
+
Setup .rubocop.yml
|
|
9
12
|
|
|
10
|
-
|
|
13
|
+
```sh
|
|
14
|
+
bundle exec rubocop-codetakt init
|
|
15
|
+
```
|
|
11
16
|
|
|
12
|
-
|
|
13
|
-
|
|
17
|
+
`init` generate the following directive to your `.rubocop.yml`:
|
|
18
|
+
|
|
19
|
+
```yaml
|
|
20
|
+
inherit_gem:
|
|
21
|
+
rubocop-codetakt:
|
|
22
|
+
- "config/rubocop.yml"
|
|
23
|
+
# uncomment if use performance cops
|
|
24
|
+
# - "config/performance.yml"
|
|
25
|
+
# uncomment if use rails cops
|
|
26
|
+
# - "config/rails.yml"
|
|
27
|
+
# uncomment if use rspec cops
|
|
28
|
+
# - "config/rspec.yml"
|
|
29
|
+
|
|
30
|
+
AllCops:
|
|
31
|
+
TargetRubyVersion: 2.5
|
|
14
32
|
```
|
|
15
33
|
|
|
16
|
-
|
|
34
|
+
```sh
|
|
35
|
+
bundle exec rubocop <options...>
|
|
36
|
+
```
|
|
37
|
+
|
|
38
|
+
## Installation
|
|
39
|
+
|
|
40
|
+
Add this line to your application's Gemfile:
|
|
17
41
|
|
|
18
42
|
```ruby
|
|
19
43
|
group :development do
|
|
20
|
-
gem
|
|
44
|
+
gem "rubocop-codetakt", require: false
|
|
21
45
|
end
|
|
22
46
|
```
|
|
23
47
|
|
|
24
|
-
|
|
48
|
+
## Contributing
|
|
49
|
+
|
|
50
|
+
Bug reports and pull requests are welcome on GitHub at https://github.com/codetakt/rubocop-codetakt.
|
|
25
51
|
|
|
26
|
-
```yaml
|
|
27
|
-
inherit_gem:
|
|
28
|
-
rubocop-codetakt:
|
|
29
|
-
- config/rubocop.yml
|
|
30
|
-
- config/rails.yml # optional.
|
|
31
|
-
- config/rspec.yml # optional.
|
|
32
|
-
```
|
|
33
52
|
|
|
34
53
|
## License
|
|
35
54
|
|
|
36
55
|
The gem is available as open source under the terms of the [MIT License](http://opensource.org/licenses/MIT).
|
|
37
|
-
|
|
38
|
-
[1]: https://github.com/onk/onkcop
|
data/Rakefile
ADDED
data/bin/console
ADDED
|
@@ -0,0 +1,14 @@
|
|
|
1
|
+
#!/usr/bin/env ruby
|
|
2
|
+
|
|
3
|
+
require "bundler/setup"
|
|
4
|
+
require "rubocop_codetakt"
|
|
5
|
+
|
|
6
|
+
# You can add fixtures and/or initialization code here to make experimenting
|
|
7
|
+
# with your gem easier. You can also use a different console, if you like.
|
|
8
|
+
|
|
9
|
+
# (If you use this, don't forget to add pry to your Gemfile!)
|
|
10
|
+
# require "pry"
|
|
11
|
+
# Pry.start
|
|
12
|
+
|
|
13
|
+
require "irb"
|
|
14
|
+
IRB.start
|
data/bin/setup
ADDED
data/config/rails.yml
CHANGED
|
@@ -1,6 +1,7 @@
|
|
|
1
|
-
|
|
2
|
-
|
|
3
|
-
|
|
1
|
+
require: "rubocop-rails"
|
|
2
|
+
|
|
3
|
+
Rails:
|
|
4
|
+
Enabled: true
|
|
4
5
|
|
|
5
6
|
# Migrations are must not depend to the ApplicationRecord,
|
|
6
7
|
# because the AR is mutable.
|
|
@@ -8,3 +9,52 @@ inherit_gem:
|
|
|
8
9
|
Rails/ApplicationRecord:
|
|
9
10
|
Exclude:
|
|
10
11
|
- "db/migrate/*.rb"
|
|
12
|
+
|
|
13
|
+
# ActiveRecord で association を安易に delegate すると
|
|
14
|
+
# N+1 を起こしまくるので、なるべく delegate 的なメソッドを使わずに
|
|
15
|
+
# コストが掛かっていることを自覚できるようにしておきたい。
|
|
16
|
+
# メソッドでも危ういが、DSL だと更に意識から抜けるので無効に。
|
|
17
|
+
Rails/Delegate:
|
|
18
|
+
Enabled: false
|
|
19
|
+
|
|
20
|
+
# 意図せずに exit を書くこと無いでしょ？
|
|
21
|
+
# 毎回 Exclude / rubocop:disable する方が手間。
|
|
22
|
+
Rails/Exit:
|
|
23
|
+
Enabled: false
|
|
24
|
+
|
|
25
|
+
# -File.join(Rails.root, "app", "models")
|
|
26
|
+
# +Rails.root.join("app", "models")
|
|
27
|
+
# はともかく
|
|
28
|
+
# -Rails.root.join("app/models")
|
|
29
|
+
# +Rails.root.join("app", "models")
|
|
30
|
+
# は Pathname#plus が行っているので意味無いのでは？
|
|
31
|
+
Rails/FilePath:
|
|
32
|
+
Enabled: false
|
|
33
|
+
|
|
34
|
+
# 桁が揃わなくて気持ち悪い
|
|
35
|
+
# create(:user, logged_in_at: 1.day.ago)
|
|
36
|
+
# create(:user, logged_in_at: 2.days.ago)
|
|
37
|
+
Rails/PluralizationGrammar:
|
|
38
|
+
Enabled: false
|
|
39
|
+
|
|
40
|
+
# unless 文を使ってでも「空」を条件にした方が
|
|
41
|
+
# 「存在する」よりも「空」の方が状態として特別なので
|
|
42
|
+
# 脳内モデルと合致しやすい。
|
|
43
|
+
Rails/Present:
|
|
44
|
+
Enabled: false
|
|
45
|
+
|
|
46
|
+
# method_missing を隠したい場合は respond_to? を使うべき
|
|
47
|
+
Rails/SafeNavigation:
|
|
48
|
+
ConvertTry: true
|
|
49
|
+
|
|
50
|
+
# valid? チェックし忘れを防ぎたい
|
|
51
|
+
Rails/SaveBang:
|
|
52
|
+
Enabled: true
|
|
53
|
+
|
|
54
|
+
# staging 環境を使っているので追加
|
|
55
|
+
Rails/UnknownEnv:
|
|
56
|
+
Environments:
|
|
57
|
+
- development # rubocop default.yml
|
|
58
|
+
- test # rubocop default.yml
|
|
59
|
+
- production # rubocop default.yml
|
|
60
|
+
- staging
|
data/config/rspec.yml
CHANGED
|
@@ -1,20 +1,73 @@
|
|
|
1
|
-
|
|
2
|
-
onkcop:
|
|
3
|
-
- config/rspec.yml
|
|
1
|
+
require: "rubocop-rspec"
|
|
4
2
|
|
|
3
|
+
# 日本語だと「〜の場合」になるので suffix でないと対応できない
|
|
4
|
+
RSpec/ContextWording:
|
|
5
|
+
Enabled: false
|
|
6
|
+
|
|
7
|
+
# subject はコピペ可搬性よりもそのまま USAGE であって欲しい
|
|
5
8
|
RSpec/DescribedClass:
|
|
9
|
+
EnforcedStyle: explicit
|
|
6
10
|
Exclude:
|
|
7
11
|
- "spec/requests/*"
|
|
8
12
|
|
|
9
|
-
#
|
|
10
|
-
#
|
|
11
|
-
#
|
|
12
|
-
#
|
|
13
|
-
#
|
|
13
|
+
# it が一つしか無いような context では空行を開ける方が読みづらい
|
|
14
|
+
# context "when foo is bar" do
|
|
15
|
+
# let(:foo) { bar }
|
|
16
|
+
# it { is_expected.to do_something }
|
|
17
|
+
# end
|
|
18
|
+
RSpec/EmptyLineAfterFinalLet:
|
|
19
|
+
Enabled: false
|
|
20
|
+
|
|
21
|
+
# each で回したり aggregate_failures 使ってたりすると厳しい。
|
|
22
|
+
# feature spec は exclude でも良いかもしれない。
|
|
23
|
+
# ヒアドキュメント使うと一瞬で超えるので disable も検討。
|
|
24
|
+
RSpec/ExampleLength:
|
|
25
|
+
Max: 8
|
|
26
|
+
|
|
27
|
+
# block の方がテスト対象が
|
|
28
|
+
# * `{}` の前後のスペースと相まって目立つ
|
|
29
|
+
# * 普段書く形と同じなので自然に脳内に入ってくる
|
|
30
|
+
RSpec/ExpectChange:
|
|
31
|
+
EnforcedStyle: block
|
|
32
|
+
|
|
33
|
+
# one-liner の should は書きやすいし意味が通りやすいし副作用も無いので撥ねる必要がない。
|
|
34
|
+
# ただ expect 派に対して強制するほどでもないので統一はしない。
|
|
35
|
+
RSpec/ImplicitExpect:
|
|
36
|
+
Enabled: false
|
|
37
|
+
|
|
38
|
+
# let を使うのは context 間で条件が違うものが存在する時だけにしたい。
|
|
39
|
+
# before の方が事前条件を整えていることが分かりやすい。
|
|
40
|
+
RSpec/InstanceVariable:
|
|
41
|
+
Enabled: false
|
|
42
|
+
|
|
43
|
+
# spec_helper で meta[:aggregate_failures] を設定することで
|
|
44
|
+
# aggregate_failures が全ての spec で有効になる。
|
|
45
|
+
#
|
|
46
|
+
# ほぼ MultipleExpectations についてはチェックされなくなる設定なので注意。
|
|
47
|
+
# パフォーマンスの問題さえ無ければ 1 example 1 assertion にしておく方が
|
|
48
|
+
# 読みやすいテストになりやすいので、そこはレビューで担保していく必要がある。
|
|
49
|
+
RSpec/MultipleExpectations:
|
|
50
|
+
AggregateFailuresByDefault: true
|
|
51
|
+
|
|
52
|
+
# 変に名前つけて呼ぶ方が分かりづらい。
|
|
53
|
+
# テスト対象メソッドを呼ぶだけの subject 以外を書かないようにする方が効く。
|
|
54
|
+
RSpec/NamedSubject:
|
|
55
|
+
Enabled: false
|
|
56
|
+
|
|
57
|
+
# Model
|
|
58
|
+
# `- #method
|
|
59
|
+
# |- 頻出ケースのテスト 1
|
|
60
|
+
# |- 頻出ケースのテスト 2
|
|
61
|
+
# `- レアケース
|
|
62
|
+
# |- レアケースのテスト 1
|
|
63
|
+
# `- レアケースのテスト 2
|
|
64
|
+
# のように括り出すと、レアケースのテストを読み飛ばせるようになり
|
|
65
|
+
# テストを読む人にやさしくなる。
|
|
66
|
+
# デフォルトの 3 より少し緩めてもヨサソウ。
|
|
14
67
|
RSpec/NestedGroups:
|
|
15
68
|
Max: 4
|
|
16
69
|
|
|
17
|
-
#
|
|
18
|
-
#
|
|
19
|
-
RSpec/
|
|
70
|
+
# ブロックは初見だと返り値を書いていると気づけないので and_return にしたいが、
|
|
71
|
+
# ブロックの方が見た目がスッキリして見やすいので、どちらでもお好きにどうぞ。
|
|
72
|
+
RSpec/ReturnFromStub:
|
|
20
73
|
Enabled: false
|
data/config/rubocop.yml
CHANGED
|
@@ -1,40 +1,376 @@
|
|
|
1
|
-
|
|
2
|
-
|
|
3
|
-
|
|
1
|
+
# 自動生成されるものはチェック対象から除外する
|
|
2
|
+
AllCops:
|
|
3
|
+
Exclude:
|
|
4
|
+
- "node_modules/**/*" # rubocop config/default.yml
|
|
5
|
+
- "vendor/**/*" # rubocop config/default.yml
|
|
6
|
+
- "db/schema.rb"
|
|
4
7
|
|
|
5
|
-
Layout
|
|
6
|
-
EnforcedStyle: leading
|
|
8
|
+
#################### Layout ################################
|
|
7
9
|
|
|
10
|
+
# メソッドをグループ分けして書き順を揃えておくと読みやすくなる。
|
|
11
|
+
# 多少のツラミはあるかもしれない。
|
|
12
|
+
# TODO: Categories を調整することで
|
|
13
|
+
# https://github.com/pocke/rubocop-rails-order_model_declarative_methods
|
|
14
|
+
# を再現できそう。
|
|
15
|
+
Layout/ClassStructure:
|
|
16
|
+
Enabled: true
|
|
17
|
+
|
|
18
|
+
# 桁揃えが綺麗にならないことが多いので migration は除外
|
|
19
|
+
Layout/ExtraSpacing:
|
|
20
|
+
Exclude:
|
|
21
|
+
- "db/migrate/*.rb"
|
|
22
|
+
|
|
23
|
+
# special_inside_parentheses (default) と比べて
|
|
24
|
+
# * 横に長くなりづらい
|
|
25
|
+
# * メソッド名の長さが変わったときに diff が少ない
|
|
26
|
+
Layout/FirstArrayElementIndentation:
|
|
27
|
+
EnforcedStyle: consistent
|
|
28
|
+
|
|
29
|
+
# ({ と hash を開始した場合に ( の位置にインデントさせる
|
|
30
|
+
# そもそも {} が必要ない可能性が高いが Style/BracesAroundHashParameters はチェックしないことにしたので
|
|
31
|
+
Layout/FirstHashElementIndentation:
|
|
32
|
+
EnforcedStyle: consistent
|
|
33
|
+
|
|
34
|
+
# * 警告 120文字
|
|
35
|
+
# * 禁止 160文字
|
|
36
|
+
# のイメージ
|
|
37
|
+
Layout/LineLength:
|
|
38
|
+
Max: 160
|
|
39
|
+
Exclude:
|
|
40
|
+
- "db/migrate/*.rb"
|
|
41
|
+
|
|
42
|
+
# This is a port from our history.
|
|
43
|
+
# Unfortunately, that process has been lost.
|
|
8
44
|
Layout/MultilineMethodCallIndentation:
|
|
9
45
|
EnforcedStyle: indented
|
|
10
46
|
IndentationWidth: 2
|
|
11
47
|
|
|
48
|
+
# {} は 1 行で書くときに主に使われるので、スペースよりも
|
|
49
|
+
# 横に長くならない方が嬉しさが多い。
|
|
50
|
+
# そもそも {| のスタイルの方が一般的だったと認識している。
|
|
51
|
+
Layout/SpaceInsideBlockBraces:
|
|
52
|
+
SpaceBeforeBlockParameters: false
|
|
53
|
+
|
|
54
|
+
#################### Lint ##################################
|
|
55
|
+
|
|
56
|
+
# spec 内では
|
|
57
|
+
# expect { subject }.to change { foo }
|
|
58
|
+
# という書き方をよく行うので () を省略したい。
|
|
59
|
+
# { foo } は明らかに change に紐付く。
|
|
60
|
+
Lint/AmbiguousBlockAssociation:
|
|
61
|
+
Exclude:
|
|
62
|
+
- "spec/**/*_spec.rb"
|
|
63
|
+
|
|
64
|
+
# Style/EmptyCaseCondition と同じく網羅の表現力が empty when を認めた方が高いし、
|
|
65
|
+
# 頻出する対象を最初の when で撥ねるのはパフォーマンス向上で頻出する。
|
|
66
|
+
# また、
|
|
67
|
+
# case foo
|
|
68
|
+
# when 42
|
|
69
|
+
# # nop
|
|
70
|
+
# when 1..100
|
|
71
|
+
# ...
|
|
72
|
+
# end
|
|
73
|
+
# と、下の when がキャッチしてしまう場合等に対応していない。
|
|
74
|
+
# See. http://tech.sideci.com/entry/2016/11/01/105900
|
|
75
|
+
Lint/EmptyWhen:
|
|
76
|
+
Enabled: false
|
|
77
|
+
|
|
78
|
+
# RuntimeError は「特定の Error を定義できない場合」なので、
|
|
79
|
+
# 定義できるエラーは RuntimeError ではなく StandardError を継承する。
|
|
80
|
+
Lint/InheritException:
|
|
81
|
+
EnforcedStyle: standard_error
|
|
82
|
+
|
|
83
|
+
# * 同名のメソッドがある場合にローカル変数に `_` を付ける
|
|
84
|
+
# * 一時変数として `_` を付ける
|
|
85
|
+
# というテクニックは頻出する
|
|
86
|
+
Lint/UnderscorePrefixedVariableName:
|
|
87
|
+
Enabled: false
|
|
88
|
+
|
|
89
|
+
# 子クラスで実装させるつもりで中身が
|
|
90
|
+
# raise NotImplementedError
|
|
91
|
+
# のみのメソッドが引っかかるので。
|
|
92
|
+
# (raise せずに中身が空だと IgnoreEmptyMethods でセーフ)
|
|
93
|
+
Lint/UnusedMethodArgument:
|
|
94
|
+
Enabled: false
|
|
95
|
+
|
|
96
|
+
# select 以外では引っかからないと思うので
|
|
97
|
+
# mutating_methods のチェックを有効に。
|
|
98
|
+
# TODO: select は引数が無い (ブロックのみ) の場合にだけチェックする
|
|
99
|
+
# ようにすると誤検知がほぼ無くなる？
|
|
100
|
+
Lint/Void:
|
|
101
|
+
CheckForMethodsWithNoSideEffects: true
|
|
102
|
+
|
|
103
|
+
#################### Metrics ###############################
|
|
104
|
+
|
|
105
|
+
# 30 まではギリギリ許せる範囲だったけど
|
|
106
|
+
# リリースごとに 3 ずつぐらい下げていきます。20 まで下げたい。
|
|
12
107
|
Metrics/AbcSize:
|
|
13
|
-
Max: 30
|
|
108
|
+
Max: 30 # Loosen for our firm.
|
|
14
109
|
|
|
110
|
+
# Gemfile, Guardfile は DSL 的で基本的に複雑にはならないので除外
|
|
111
|
+
# rake, rspec, environments, routes は巨大な block 不可避なので除外
|
|
112
|
+
# TODO: ExcludedMethods の精査
|
|
113
|
+
Metrics/BlockLength:
|
|
114
|
+
Exclude:
|
|
115
|
+
- "Rakefile"
|
|
116
|
+
- "**/*.rake"
|
|
117
|
+
- "spec/**/*.rb"
|
|
118
|
+
- "Gemfile"
|
|
119
|
+
- "Guardfile"
|
|
120
|
+
- "config/environments/*.rb"
|
|
121
|
+
- "config/routes.rb"
|
|
122
|
+
- "config/routes/**/*.rb"
|
|
123
|
+
- "*.gemspec"
|
|
124
|
+
|
|
125
|
+
# 6 は強すぎるので緩める
|
|
15
126
|
Metrics/CyclomaticComplexity:
|
|
16
127
|
Max: 10
|
|
17
128
|
|
|
129
|
+
# 20 行超えるのは migration ファイル以外滅多に無い
|
|
18
130
|
Metrics/MethodLength:
|
|
19
131
|
Max: 20
|
|
132
|
+
Exclude:
|
|
133
|
+
- "db/migrate/*.rb"
|
|
20
134
|
|
|
135
|
+
# 分岐の数。ガード句を多用しているとデフォルト 7 だと厳しい
|
|
21
136
|
Metrics/PerceivedComplexity:
|
|
22
|
-
Max: 10
|
|
137
|
+
Max: 10 # Loosen for our firm.
|
|
138
|
+
|
|
139
|
+
|
|
140
|
+
#################### Naming ################################
|
|
141
|
+
|
|
142
|
+
# has_ から始まるメソッドは許可する
|
|
143
|
+
Naming/PredicateName:
|
|
144
|
+
ForbiddenPrefixes:
|
|
145
|
+
- "is_"
|
|
146
|
+
- "have_"
|
|
147
|
+
NamePrefix:
|
|
148
|
+
- "is_"
|
|
149
|
+
- "have_"
|
|
150
|
+
|
|
151
|
+
# 3 文字未満だと指摘されるが、未使用を示す _ や e(rror), b(lock),
|
|
152
|
+
# n(umber) といった 1 文字変数は頻出するし、前置詞(by, to, ...)や
|
|
153
|
+
# よく知られた省略語 (op: operator とか pk: primary key とか) も妥当。
|
|
154
|
+
# 変数 s にどんな文字列かを形容したい場合と、不要な場合とがある＝無効
|
|
155
|
+
Naming/MethodParameterName:
|
|
156
|
+
Enabled: false
|
|
157
|
+
|
|
158
|
+
#################### Security ##############################
|
|
159
|
+
|
|
160
|
+
# 毎回 YAML.safe_load(yaml_str, [Date, Time]) するのは面倒で。。
|
|
161
|
+
Security/YAMLLoad:
|
|
162
|
+
Enabled: false
|
|
163
|
+
|
|
164
|
+
|
|
165
|
+
#################### Style #################################
|
|
166
|
+
|
|
167
|
+
# レキシカルスコープの扱いが alias_method の方が自然。
|
|
168
|
+
# https://ernie.io/2014/10/23/in-defense-of-alias/ のように
|
|
169
|
+
# 問題になる場合は自分で緩める。
|
|
170
|
+
Style/Alias:
|
|
171
|
+
EnforcedStyle: prefer_alias_method
|
|
172
|
+
|
|
173
|
+
# redirect_to xxx and return のイディオムを維持したい
|
|
174
|
+
Style/AndOr:
|
|
175
|
+
EnforcedStyle: conditionals
|
|
176
|
+
|
|
177
|
+
# 日本語のコメントを許可する
|
|
178
|
+
Style/AsciiComments:
|
|
179
|
+
Enabled: false
|
|
180
|
+
|
|
181
|
+
# do .. end から更にメソッドチェーンすると見づらいので
|
|
182
|
+
# auto-correct せず、自分で修正する
|
|
183
|
+
# spec 内は見た目が綺麗になるので許可
|
|
184
|
+
Style/BlockDelimiters:
|
|
185
|
+
AutoCorrect: false
|
|
186
|
+
Exclude:
|
|
187
|
+
- "spec/**/*_spec.rb"
|
|
188
|
+
|
|
189
|
+
# scope が違うとか親 module の存在確認が必要とかデメリットはあるが、
|
|
190
|
+
# namespace 付きのクラスはかなり頻繁に作るので簡単に書きたい。
|
|
191
|
+
Style/ClassAndModuleChildren:
|
|
192
|
+
Enabled: false
|
|
193
|
+
|
|
194
|
+
# Style/CollectionMethods 自体は無効になっているのだが、
|
|
195
|
+
# https://github.com/bbatsov/rubocop/issues/1084
|
|
196
|
+
# https://github.com/bbatsov/rubocop/issues/1334
|
|
197
|
+
# Performance/Detect がこの設定値を見るので PreferredMethods だけ変更しておく。
|
|
198
|
+
#
|
|
199
|
+
# デフォルト値から変えたのは
|
|
200
|
+
# find -> detect
|
|
201
|
+
# ActiveRecord の find と間違えやすいため
|
|
202
|
+
# reduce -> inject
|
|
203
|
+
# detect, reject, select と並べたときに韻を踏んでいるため。
|
|
204
|
+
# collect -> map を維持しているのは文字数が圧倒的に少ないため。
|
|
205
|
+
Style/CollectionMethods:
|
|
206
|
+
PreferredMethods:
|
|
207
|
+
detect: "detect"
|
|
208
|
+
find: "detect"
|
|
209
|
+
inject: "inject"
|
|
210
|
+
reduce: "inject"
|
|
211
|
+
|
|
212
|
+
# ドキュメントの無い public class を許可する
|
|
213
|
+
Style/Documentation:
|
|
214
|
+
Enabled: false
|
|
215
|
+
|
|
216
|
+
# !! のイディオムは積極的に使う
|
|
217
|
+
Style/DoubleNegation:
|
|
218
|
+
Enabled: false
|
|
219
|
+
|
|
220
|
+
# case
|
|
221
|
+
# when ios?
|
|
222
|
+
# when android?
|
|
223
|
+
# end
|
|
224
|
+
# のようなものは case の方が網羅の表現力が高い
|
|
225
|
+
Style/EmptyCaseCondition:
|
|
226
|
+
Enabled: false
|
|
227
|
+
|
|
228
|
+
# 明示的に else で nil を返すのは分かりやすいので許可する
|
|
229
|
+
Style/EmptyElse:
|
|
230
|
+
EnforcedStyle: empty
|
|
231
|
+
|
|
232
|
+
# 空メソッドの場合だけ1行で書かなければいけない理由が無い
|
|
233
|
+
# 「セミコロンは使わない」に寄せた方がルールがシンプル
|
|
234
|
+
Style/EmptyMethod:
|
|
235
|
+
EnforcedStyle: expanded
|
|
236
|
+
|
|
237
|
+
# いずれかに揃えるのならば `sprintf` や `format` より String#% が好きです
|
|
238
|
+
Style/FormatString:
|
|
239
|
+
EnforcedStyle: percent
|
|
240
|
+
|
|
241
|
+
# まだ対応するには早い
|
|
242
|
+
Style/FrozenStringLiteralComment:
|
|
243
|
+
Enabled: false
|
|
244
|
+
|
|
245
|
+
# if 文の中に 3 行程度のブロックを書くぐらいは許容した方が現実的
|
|
246
|
+
# NOTE: https://github.com/bbatsov/rubocop/commit/29945958034db13af9e8ff385ec58cb9eb464596
|
|
247
|
+
# の影響で、if 文の中身が 1 行の場合に警告されるようになっている。
|
|
248
|
+
# Style/IfUnlessModifier の設定見てくれないかなぁ？ (v0.36.0)
|
|
249
|
+
Style/GuardClause:
|
|
250
|
+
MinBodyLength: 5
|
|
251
|
+
|
|
252
|
+
# rake タスクの順序の hash は rocket を許可する
|
|
253
|
+
Style/HashSyntax:
|
|
254
|
+
Exclude:
|
|
255
|
+
- "**/*.rake"
|
|
256
|
+
- "Rakefile"
|
|
257
|
+
|
|
258
|
+
# 平たくしてしまうと条件のグルーピングが脳内モデルとズレやすい
|
|
259
|
+
Style/IfInsideElse:
|
|
260
|
+
Enabled: false
|
|
261
|
+
|
|
262
|
+
# 条件式の方を意識させたい場合には後置の if/unless を使わない方が分かりやすい
|
|
263
|
+
Style/IfUnlessModifier:
|
|
264
|
+
Enabled: false
|
|
265
|
+
|
|
266
|
+
# scope 等は複数行でも lambda ではなく ->{} で揃えた方が見た目が綺麗
|
|
267
|
+
Style/Lambda:
|
|
268
|
+
EnforcedStyle: literal
|
|
269
|
+
|
|
270
|
+
# この 2 つは単発で動かすのが分かっているので Object を汚染しても問題ない。
|
|
271
|
+
# spec/dummy は Rails Engine を開発するときに絶対に引っかかるので入れておく。
|
|
272
|
+
Style/MixinUsage:
|
|
273
|
+
Exclude:
|
|
274
|
+
- "bin/setup"
|
|
275
|
+
- "bin/update"
|
|
276
|
+
- "spec/dummy/bin/setup"
|
|
277
|
+
- "spec/dummy/bin/update"
|
|
278
|
+
|
|
279
|
+
# 1_000_000 と区切り文字が 2 個以上必要になる場合のみ _ 区切りを必須にする
|
|
280
|
+
# 10_000_00 は許可しない。(これは例えば 10000 ドルをセント単位にする時に便利だが
|
|
281
|
+
# 頻出しないので foolproof に振る
|
|
282
|
+
Style/NumericLiterals:
|
|
283
|
+
MinDigits: 7
|
|
284
|
+
Strict: true
|
|
23
285
|
|
|
24
|
-
|
|
25
|
-
|
|
286
|
+
# foo.positive? は foo > 0 に比べて意味が曖昧になる
|
|
287
|
+
# foo.zero? は許可したいけどメソッドごとに指定できないので一括で disable に
|
|
288
|
+
Style/NumericPredicate:
|
|
289
|
+
Enabled: false
|
|
290
|
+
|
|
291
|
+
# falsy な場合という条件式の方を意識させたい場合がある。
|
|
292
|
+
# Style/IfUnlessModifier と同じ雰囲気。
|
|
293
|
+
Style/OrAssignment:
|
|
294
|
+
Enabled: false
|
|
295
|
+
|
|
296
|
+
# 正規表現にマッチさせた時の特殊変数の置き換えは Regex.last_match ではなく
|
|
297
|
+
# 名前付きキャプチャを使って参照したいので auto-correct しない
|
|
298
|
+
Style/PerlBackrefs:
|
|
299
|
+
AutoCorrect: false
|
|
300
|
+
|
|
301
|
+
# Hash#has_key? の方が key? よりも意味が通る
|
|
302
|
+
Style/PreferredHashMethods:
|
|
303
|
+
EnforcedStyle: verbose
|
|
26
304
|
|
|
27
|
-
|
|
305
|
+
# 受け取り側で multiple assignment しろというのを明示
|
|
306
|
+
Style/RedundantReturn:
|
|
307
|
+
AllowMultipleReturnValues: true
|
|
308
|
+
|
|
309
|
+
# 特に model 内において、ローカル変数とメソッド呼び出しの区別をつけた方が分かりやすい場合が多い
|
|
310
|
+
Style/RedundantSelf:
|
|
28
311
|
Enabled: false
|
|
29
312
|
|
|
30
|
-
|
|
31
|
-
|
|
313
|
+
# 無指定だと StandardError を rescue するのは常識の範疇なので。
|
|
314
|
+
Style/RescueStandardError:
|
|
315
|
+
EnforcedStyle: implicit
|
|
316
|
+
|
|
317
|
+
# user&.admin? が、[nil, true, false] の 3 値を返すことに一瞬で気づけず
|
|
318
|
+
# boolean を返すっぽく見えてしまうので無効に。
|
|
319
|
+
# user && user.admin? なら短絡評価で nil が返ってくるのが一目で分かるので。
|
|
320
|
+
# (boolean を返すメソッド以外なら積極的に使いたいんだけどねぇ
|
|
321
|
+
#
|
|
322
|
+
# 他に auto-correct してはいけないパターンとして
|
|
323
|
+
# if hoge && hoge.count > 1
|
|
324
|
+
# がある。
|
|
325
|
+
Style/SafeNavigation:
|
|
326
|
+
Enabled: false
|
|
32
327
|
|
|
33
|
-
|
|
34
|
-
|
|
328
|
+
# spec 内は見た目が綺麗になるので許可
|
|
329
|
+
Style/Semicolon:
|
|
330
|
+
Exclude:
|
|
331
|
+
- "spec/**/*_spec.rb"
|
|
35
332
|
|
|
333
|
+
# String#intern は ruby の内部表現すぎるので String#to_sym を使う
|
|
334
|
+
Style/StringMethods:
|
|
335
|
+
Enabled: true
|
|
336
|
+
|
|
337
|
+
# %w() と %i() が見分けづらいので Style/WordArray と合わせて無効に。
|
|
338
|
+
# 書き手に委ねるという意味で、Enabled: false にしています。使っても良い。
|
|
339
|
+
Style/SymbolArray:
|
|
340
|
+
Enabled: false
|
|
341
|
+
|
|
342
|
+
# 三項演算子は分かりやすく使いたい。
|
|
343
|
+
# () を外さない方が条件式が何なのか読み取りやすいと感じる。
|
|
344
|
+
Style/TernaryParentheses:
|
|
345
|
+
EnforcedStyle: require_parentheses_when_complex
|
|
346
|
+
|
|
347
|
+
# Count on developers the handling of the trailing commas
|
|
348
|
+
# Make consistency with the eslint-config-codetakt.
|
|
36
349
|
Style/TrailingCommaInArguments:
|
|
37
350
|
Enabled: false
|
|
38
351
|
|
|
39
|
-
Style/
|
|
352
|
+
Style/TrailingCommaInArrayLiteral:
|
|
353
|
+
Enabled: false
|
|
354
|
+
|
|
355
|
+
Style/TrailingCommaInHashLiteral:
|
|
356
|
+
Enabled: false
|
|
357
|
+
|
|
358
|
+
# %w() と %i() が見分けづらいので Style/SymbolArray と合わせて無効に。
|
|
359
|
+
# 書き手に委ねるという意味で、Enabled: false にしています。使っても良い。
|
|
360
|
+
Style/WordArray:
|
|
361
|
+
Enabled: false
|
|
362
|
+
|
|
363
|
+
# 0 <= foo && foo < 5 のように数直線上に並べるのは
|
|
364
|
+
# コードを読みやすくするテクニックなので equality_operators_only に。
|
|
365
|
+
# Style/YodaCondition:
|
|
366
|
+
# # TODO: rubocop 0.63.0以降はforbid_for_equality_operators_onlyなので依存を引き上げれば有効にできる
|
|
367
|
+
# EnforcedStyle: forbid_for_equality_operators_only
|
|
368
|
+
|
|
369
|
+
# 条件式で arr.size > 0 が使われた時に
|
|
370
|
+
# if !arr.empty?
|
|
371
|
+
# else
|
|
372
|
+
# end
|
|
373
|
+
# に修正されるのが嫌。
|
|
374
|
+
# 中身を入れ替えて否定外しても良いんだけど、どちらが例外的な処理なのかが分かりづらくなる。
|
|
375
|
+
Style/ZeroLengthPredicate:
|
|
40
376
|
Enabled: false
|
|
@@ -0,0 +1,46 @@
|
|
|
1
|
+
require "fileutils"
|
|
2
|
+
|
|
3
|
+
module RuboCop
|
|
4
|
+
module Codetakt
|
|
5
|
+
class CLI
|
|
6
|
+
def self.start(args)
|
|
7
|
+
action_name = retrieve_command_name(args)
|
|
8
|
+
unless action_name
|
|
9
|
+
print_help
|
|
10
|
+
exit
|
|
11
|
+
end
|
|
12
|
+
|
|
13
|
+
instance = self.new
|
|
14
|
+
if instance.public_methods(false).include?(action_name.to_sym)
|
|
15
|
+
instance.__send__(action_name, args)
|
|
16
|
+
exit
|
|
17
|
+
end
|
|
18
|
+
|
|
19
|
+
puts "Could not find command #{action_name}."
|
|
20
|
+
print_help
|
|
21
|
+
exit(1)
|
|
22
|
+
rescue => e
|
|
23
|
+
puts e.message
|
|
24
|
+
exit(1)
|
|
25
|
+
end
|
|
26
|
+
|
|
27
|
+
def self.retrieve_command_name(args)
|
|
28
|
+
meth = args.first.to_s unless args.empty?
|
|
29
|
+
args.shift if meth && (meth !~ /^\-/)
|
|
30
|
+
end
|
|
31
|
+
|
|
32
|
+
def self.print_help
|
|
33
|
+
puts "rubocop-codetakt commands:"
|
|
34
|
+
puts " init - Setup .rubocop.yml"
|
|
35
|
+
end
|
|
36
|
+
|
|
37
|
+
CONFIG_FILE_NAME = ".rubocop.yml"
|
|
38
|
+
def init(args)
|
|
39
|
+
raise "usage: rubocop-codetakt init" unless args.empty?
|
|
40
|
+
template_path = File.expand_path("../../../templates", __dir__)
|
|
41
|
+
puts "#{File.exist?(CONFIG_FILE_NAME) ? "overwrite" : "create"} #{CONFIG_FILE_NAME}"
|
|
42
|
+
FileUtils.copy_file(File.join(template_path, CONFIG_FILE_NAME), CONFIG_FILE_NAME)
|
|
43
|
+
end
|
|
44
|
+
end
|
|
45
|
+
end
|
|
46
|
+
end
|
|
@@ -0,0 +1,25 @@
|
|
|
1
|
+
lib = File.expand_path("lib", __dir__)
|
|
2
|
+
$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
|
|
3
|
+
require "rubocop/codetakt/version"
|
|
4
|
+
|
|
5
|
+
Gem::Specification.new do |spec|
|
|
6
|
+
spec.name = "rubocop-codetakt"
|
|
7
|
+
spec.version = RuboCop::Codetakt::VERSION
|
|
8
|
+
spec.authors = ["codeTakt Developers"]
|
|
9
|
+
spec.email = ["dev@codetakt.com"]
|
|
10
|
+
|
|
11
|
+
spec.summary = "Unify the coding style of Ruby within codeTakt Inc."
|
|
12
|
+
spec.description = <<-EOF
|
|
13
|
+
This includes the RuboCop configuration used by codeTakt. It is
|
|
14
|
+
for the unification and linting of coding styles.
|
|
15
|
+
EOF
|
|
16
|
+
spec.homepage = "https://github.com/codetakt/rubocop-codetakt"
|
|
17
|
+
spec.license = "MIT"
|
|
18
|
+
|
|
19
|
+
spec.files = `git ls-files -z`.split("\x0").reject {|f| f.match(%r{^(test|spec|features)/}) }
|
|
20
|
+
spec.bindir = "exe"
|
|
21
|
+
spec.executables = spec.files.grep(%r{^exe/}) {|f| File.basename(f) }
|
|
22
|
+
spec.require_paths = ["lib"]
|
|
23
|
+
|
|
24
|
+
spec.add_dependency "rubocop", "~> 0.82.0"
|
|
25
|
+
end
|
|
@@ -0,0 +1,12 @@
|
|
|
1
|
+
inherit_gem:
|
|
2
|
+
rubocop-codetakt:
|
|
3
|
+
- "config/rubocop.yml"
|
|
4
|
+
# uncomment if use performance cops
|
|
5
|
+
# - "config/performance.yml"
|
|
6
|
+
# uncomment if use rails cops
|
|
7
|
+
# - "config/rails.yml"
|
|
8
|
+
# uncomment if use rspec cops
|
|
9
|
+
# - "config/rspec.yml"
|
|
10
|
+
|
|
11
|
+
AllCops:
|
|
12
|
+
TargetRubyVersion: 2.5
|
metadata
CHANGED
|
@@ -1,70 +1,62 @@
|
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
|
2
2
|
name: rubocop-codetakt
|
|
3
3
|
version: !ruby/object:Gem::Version
|
|
4
|
-
version: 0.
|
|
4
|
+
version: 0.82.0.0
|
|
5
5
|
platform: ruby
|
|
6
6
|
authors:
|
|
7
|
-
- codeTakt
|
|
7
|
+
- codeTakt Developers
|
|
8
8
|
autorequire:
|
|
9
|
-
bindir:
|
|
9
|
+
bindir: exe
|
|
10
10
|
cert_chain: []
|
|
11
|
-
date:
|
|
11
|
+
date: 2020-04-20 00:00:00.000000000 Z
|
|
12
12
|
dependencies:
|
|
13
13
|
- !ruby/object:Gem::Dependency
|
|
14
|
-
name:
|
|
15
|
-
requirement: !ruby/object:Gem::Requirement
|
|
16
|
-
requirements:
|
|
17
|
-
- - '='
|
|
18
|
-
- !ruby/object:Gem::Version
|
|
19
|
-
version: 0.52.1.1
|
|
20
|
-
type: :runtime
|
|
21
|
-
prerelease: false
|
|
22
|
-
version_requirements: !ruby/object:Gem::Requirement
|
|
23
|
-
requirements:
|
|
24
|
-
- - '='
|
|
25
|
-
- !ruby/object:Gem::Version
|
|
26
|
-
version: 0.52.1.1
|
|
27
|
-
- !ruby/object:Gem::Dependency
|
|
28
|
-
name: bundler
|
|
29
|
-
requirement: !ruby/object:Gem::Requirement
|
|
30
|
-
requirements:
|
|
31
|
-
- - "~>"
|
|
32
|
-
- !ruby/object:Gem::Version
|
|
33
|
-
version: '1.14'
|
|
34
|
-
type: :development
|
|
35
|
-
prerelease: false
|
|
36
|
-
version_requirements: !ruby/object:Gem::Requirement
|
|
37
|
-
requirements:
|
|
38
|
-
- - "~>"
|
|
39
|
-
- !ruby/object:Gem::Version
|
|
40
|
-
version: '1.14'
|
|
41
|
-
- !ruby/object:Gem::Dependency
|
|
42
|
-
name: rake
|
|
14
|
+
name: rubocop
|
|
43
15
|
requirement: !ruby/object:Gem::Requirement
|
|
44
16
|
requirements:
|
|
45
17
|
- - "~>"
|
|
46
18
|
- !ruby/object:Gem::Version
|
|
47
|
-
version:
|
|
48
|
-
type: :
|
|
19
|
+
version: 0.82.0
|
|
20
|
+
type: :runtime
|
|
49
21
|
prerelease: false
|
|
50
22
|
version_requirements: !ruby/object:Gem::Requirement
|
|
51
23
|
requirements:
|
|
52
24
|
- - "~>"
|
|
53
25
|
- !ruby/object:Gem::Version
|
|
54
|
-
version:
|
|
55
|
-
description:
|
|
56
|
-
|
|
57
|
-
|
|
26
|
+
version: 0.82.0
|
|
27
|
+
description: |2
|
|
28
|
+
This includes the RuboCop configuration used by codeTakt. It is
|
|
29
|
+
for the unification and linting of coding styles.
|
|
30
|
+
email:
|
|
31
|
+
- dev@codetakt.com
|
|
32
|
+
executables:
|
|
33
|
+
- rubocop-codetakt
|
|
58
34
|
extensions: []
|
|
59
35
|
extra_rdoc_files: []
|
|
60
36
|
files:
|
|
37
|
+
- ".gitignore"
|
|
38
|
+
- ".rspec"
|
|
39
|
+
- ".rubocop.yml"
|
|
40
|
+
- ".travis.yml"
|
|
61
41
|
- CHANGELOG.md
|
|
62
|
-
-
|
|
42
|
+
- Gemfile
|
|
43
|
+
- LICENSE.txt
|
|
63
44
|
- README.md
|
|
45
|
+
- Rakefile
|
|
46
|
+
- bin/console
|
|
47
|
+
- bin/setup
|
|
48
|
+
- config/performance.yml
|
|
64
49
|
- config/rails.yml
|
|
65
50
|
- config/rspec.yml
|
|
66
51
|
- config/rubocop.yml
|
|
67
|
-
|
|
52
|
+
- exe/rubocop-codetakt
|
|
53
|
+
- lib/rubocop/codetakt.rb
|
|
54
|
+
- lib/rubocop/codetakt/cli.rb
|
|
55
|
+
- lib/rubocop/codetakt/version.rb
|
|
56
|
+
- lib/rubocop_codetakt.rb
|
|
57
|
+
- rubocop-codetakt.gemspec
|
|
58
|
+
- templates/.rubocop.yml
|
|
59
|
+
homepage: https://github.com/codetakt/rubocop-codetakt
|
|
68
60
|
licenses:
|
|
69
61
|
- MIT
|
|
70
62
|
metadata: {}
|
|
@@ -83,9 +75,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
|
|
|
83
75
|
- !ruby/object:Gem::Version
|
|
84
76
|
version: '0'
|
|
85
77
|
requirements: []
|
|
86
|
-
|
|
87
|
-
rubygems_version: 2.5.2
|
|
78
|
+
rubygems_version: 3.1.2
|
|
88
79
|
signing_key:
|
|
89
80
|
specification_version: 4
|
|
90
|
-
summary:
|
|
81
|
+
summary: Unify the coding style of Ruby within codeTakt Inc.
|
|
91
82
|
test_files: []
|