rubocop-yayoi 0.0.2 → 0.1.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/.rubocop.yml +19 -0
- data/README.md +2 -0
- data/lib/rubocop/cop/cop.rb +15 -0
- data/lib/rubocop/cop/layout/def_end_alignment.rb +1 -1
- data/lib/rubocop/cop/layout/empty_line_between_defs.rb +1 -1
- data/lib/rubocop/cop/layout/empty_lines.rb +1 -1
- data/lib/rubocop/cop/layout/empty_lines_around_method_body.rb +1 -1
- data/lib/rubocop/cop/layout/indentation_consistency.rb +1 -1
- data/lib/rubocop/cop/layout/indentation_width.rb +3 -2
- data/lib/rubocop/cop/layout/initial_indentation.rb +1 -1
- data/lib/rubocop/cop/lint/ambiguous_block_association.rb +10 -3
- data/lib/rubocop/cop/lint/ineffective_access_modifier.rb +19 -7
- data/lib/rubocop/cop/lint/unreachable_code.rb +1 -1
- data/lib/rubocop/cop/lint/unused_method_argument.rb +0 -2
- data/lib/rubocop/cop/lint/useless_access_modifier.rb +47 -1
- data/lib/rubocop/cop/lint/useless_assignment.rb +8 -1
- data/lib/rubocop/cop/metrics/cyclomatic_complexity.rb +1 -1
- data/lib/rubocop/cop/metrics/line_length.rb +1 -1
- data/lib/rubocop/cop/metrics/perceived_complexity.rb +1 -1
- data/lib/rubocop/cop/mixin/empty_lines_around_body.rb +24 -3
- data/lib/rubocop/cop/mixin/end_keyword_alignment.rb +13 -2
- data/lib/rubocop/cop/mixin/method_complexity.rb +32 -0
- data/lib/rubocop/cop/mixin/space_after_punctuation.rb +8 -1
- data/lib/rubocop/cop/mixin/uncommunicative_name.rb +9 -2
- data/lib/rubocop/cop/naming/ascii_identifiers.rb +1 -1
- data/lib/rubocop/cop/naming/class_and_module_camel_case.rb +1 -1
- data/lib/rubocop/cop/naming/variable_name.rb +1 -1
- data/lib/rubocop/cop/style/and_or.rb +2 -1
- data/lib/rubocop/cop/style/ascii_comments.rb +1 -1
- data/lib/rubocop/cop/style/colon_method_call.rb +1 -1
- data/lib/rubocop/cop/style/comment_annotation.rb +28 -3
- data/lib/rubocop/cop/style/def_with_parentheses.rb +1 -1
- data/lib/rubocop/cop/style/documentation.rb +12 -1
- data/lib/rubocop/cop/style/double_negation.rb +1 -1
- data/lib/rubocop/cop/style/empty_method.rb +6 -2
- data/lib/rubocop/cop/style/mutable_constant.rb +1 -1
- data/lib/rubocop/cop/style/numeric_literals.rb +1 -1
- data/lib/rubocop/cop/style/redundant_return.rb +11 -2
- data/lib/rubocop/cop/style/symbol_array.rb +21 -0
- data/lib/rubocop/cop/style/unneeded_condition.rb +10 -2
- data/lib/rubocop/cop/style/unneeded_interpolation.rb +1 -1
- data/lib/rubocop/cop/style/word_array.rb +11 -2
- data/lib/rubocop/yayoi/version.rb +1 -1
- data/lib/rubocop-yayoi.rb +4 -0
- metadata +5 -2
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA256:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: c8b1eb1be46ec7f4b73fbe7bceae1a6eb190f77e8603f7df5ba08018bdc14baf
|
4
|
+
data.tar.gz: a2dbade19948eacf246522e3dd3f1d8c3883e22097a674d527208529e041029d
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: 9c34b3346bb085e5a2c81779ec51dd815a0df5518534a981ce77ed012cca4dcc06feb9e21126ff241df1a50c8e0b6fc2766fddaf73af3fd60c712ef54a90bb94
|
7
|
+
data.tar.gz: 77486cb28980de30d35c1f05113710a4e9849f54dd4814c259fdba8474360a1812609271bfe6539f331e73a3431b3da461b87f35cb91473908a4b36e07a9ba9b
|
data/.rubocop.yml
CHANGED
@@ -7,6 +7,25 @@ AllCops:
|
|
7
7
|
- 'spec/fixtures/**/*'
|
8
8
|
- 'tmp/**/*'
|
9
9
|
|
10
|
+
# Offense count: 77
|
11
|
+
Metrics/AbcSize:
|
12
|
+
Max: 17
|
13
|
+
|
14
|
+
# Offense count: 52
|
15
|
+
# Configuration parameters: CountComments.
|
16
|
+
Metrics/ClassLength:
|
17
|
+
Max: 181
|
18
|
+
|
19
|
+
# Offense count: 222
|
20
|
+
# Configuration parameters: CountComments, ExcludedMethods.
|
21
|
+
Metrics/MethodLength:
|
22
|
+
Max: 14
|
23
|
+
|
24
|
+
# Offense count: 5
|
25
|
+
# Configuration parameters: CountComments.
|
26
|
+
Metrics/ModuleLength:
|
27
|
+
Max: 141
|
28
|
+
|
10
29
|
Naming/FileName:
|
11
30
|
Exclude:
|
12
31
|
- lib/rubocop-yayoi.rb
|
data/README.md
CHANGED
@@ -1,5 +1,7 @@
|
|
1
1
|
# Rubocop::Yayoi
|
2
2
|
|
3
|
+
[](https://travis-ci.org/YutaGoto/rubocop-yayoi)
|
4
|
+
|
3
5
|
`ζ*'ヮ')ζ<うっうー!`
|
4
6
|
|
5
7
|
このgemは [rubocop](https://github.com/rubocop-hq/rubocop) 実行後に表示されるテキストがアイドルマスター 765プロ所属の高槻やよい風に指摘してくれるgemです!
|
@@ -4,8 +4,9 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Layout
|
6
6
|
class IndentationWidth < Cop
|
7
|
-
|
8
|
-
|
7
|
+
YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!" \
|
8
|
+
'%<configured_indentation_width>dスペースを使いましょうねー' \
|
9
|
+
'(%<indentation>dはだめですよー)'.freeze
|
9
10
|
end
|
10
11
|
end
|
11
12
|
end
|
@@ -4,9 +4,16 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Lint
|
6
6
|
class AmbiguousBlockAssociation < Cop
|
7
|
-
|
8
|
-
|
9
|
-
|
7
|
+
YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!" \
|
8
|
+
'ブロックが`%<method>s`メソッドの呼び出しにちゃんと関連連れられているかわからないですー。' \
|
9
|
+
'ちゃんと`%<param>s`にカッコを使いましょうねー!'.freeze
|
10
|
+
|
11
|
+
def message(send_node)
|
12
|
+
block_param = send_node.last_argument
|
13
|
+
|
14
|
+
format(YAYOI_MSG, param: block_param.source,
|
15
|
+
method: block_param.send_node.source)
|
16
|
+
end
|
10
17
|
end
|
11
18
|
end
|
12
19
|
end
|
@@ -4,13 +4,25 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Lint
|
6
6
|
class IneffectiveAccessModifier < Cop
|
7
|
-
|
8
|
-
|
9
|
-
|
10
|
-
|
11
|
-
|
12
|
-
|
13
|
-
|
7
|
+
YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!%<line>d行目にある`%<modifier>s`は" \
|
8
|
+
'シングルトンメソッドの%<modifier>sを作りませんよー!' \
|
9
|
+
'代わりに%<alternative>s'.freeze
|
10
|
+
YAYOI_ALTERNATIVE_PRIVATE = '`private_class_method`か`private`を' \
|
11
|
+
'`class << self`ブロックの内部に置きましょー!'.freeze
|
12
|
+
YAYOI_ALTERNATIVE_PROTECTED = '`protected`を' \
|
13
|
+
'`class << self`ブロックの内部に置きましょー!'.freeze
|
14
|
+
|
15
|
+
def format_message(modifier)
|
16
|
+
visibility = modifier.method_name
|
17
|
+
alternative = if visibility == :private
|
18
|
+
YAYOI_ALTERNATIVE_PRIVATE
|
19
|
+
else
|
20
|
+
YAYOI_ALTERNATIVE_PROTECTED
|
21
|
+
end
|
22
|
+
format(YAYOI_MSG, modifier: visibility,
|
23
|
+
line: modifier.location.expression.line,
|
24
|
+
alternative: alternative)
|
25
|
+
end
|
14
26
|
end
|
15
27
|
end
|
16
28
|
end
|
@@ -4,7 +4,6 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Lint
|
6
6
|
class UnusedMethodArgument < Cop
|
7
|
-
# rubocop:disable Metrics/MethodLength
|
8
7
|
def message(variable)
|
9
8
|
message = String.new(
|
10
9
|
"ζ*'ヮ')ζ<うっうー!使っていない引数がありますよー `#{variable.name}`!"
|
@@ -24,7 +23,6 @@ module RuboCop
|
|
24
23
|
|
25
24
|
message
|
26
25
|
end
|
27
|
-
# rubocop:enable Metrics/MethodLength
|
28
26
|
end
|
29
27
|
end
|
30
28
|
end
|
@@ -4,7 +4,53 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Lint
|
6
6
|
class UselessAccessModifier < Cop
|
7
|
-
|
7
|
+
YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!`%<current>s`は意味がないですよー!".freeze
|
8
|
+
|
9
|
+
def check_node(node)
|
10
|
+
return if node.nil?
|
11
|
+
|
12
|
+
if node.begin_type?
|
13
|
+
check_scope(node)
|
14
|
+
elsif node.send_type? && node.bare_access_modifier?
|
15
|
+
add_offense(
|
16
|
+
node,
|
17
|
+
message: format(YAYOI_MSG, current: node.method_name)
|
18
|
+
)
|
19
|
+
end
|
20
|
+
end
|
21
|
+
|
22
|
+
def check_scope(node)
|
23
|
+
cur_vis, unused = check_child_nodes(node, nil, :public)
|
24
|
+
|
25
|
+
return unless unused
|
26
|
+
|
27
|
+
add_offense(unused, message: format(YAYOI_MSG, current: cur_vis))
|
28
|
+
end
|
29
|
+
|
30
|
+
def check_send_node(node, cur_vis, unused)
|
31
|
+
if node.bare_access_modifier?
|
32
|
+
check_new_visibility(node, unused, node.method_name, cur_vis)
|
33
|
+
elsif node.method_name == :private_class_method && !node.arguments?
|
34
|
+
add_offense(
|
35
|
+
node,
|
36
|
+
message: format(YAYOI_MSG, current: node.method_name)
|
37
|
+
)
|
38
|
+
[cur_vis, unused]
|
39
|
+
end
|
40
|
+
end
|
41
|
+
|
42
|
+
def check_new_visibility(node, unused, new_vis, cur_vis)
|
43
|
+
if new_vis == cur_vis
|
44
|
+
add_offense(node, message: format(YAYOI_MSG, current: cur_vis))
|
45
|
+
else
|
46
|
+
if unused
|
47
|
+
add_offense(unused, message: format(YAYOI_MSG, current: cur_vis))
|
48
|
+
end
|
49
|
+
unused = node
|
50
|
+
end
|
51
|
+
|
52
|
+
[new_vis, unused]
|
53
|
+
end
|
8
54
|
end
|
9
55
|
end
|
10
56
|
end
|
@@ -4,7 +4,14 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Lint
|
6
6
|
class UselessAssignment < Cop
|
7
|
-
|
7
|
+
YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!使っていない変数がありますよー! - `%<variable>s`.".freeze
|
8
|
+
|
9
|
+
def message_for_useless_assignment(assignment)
|
10
|
+
variable = assignment.variable
|
11
|
+
|
12
|
+
format(YAYOI_MSG, variable: variable.name) +
|
13
|
+
message_specification(assignment, variable).to_s
|
14
|
+
end
|
8
15
|
end
|
9
16
|
end
|
10
17
|
end
|
@@ -4,9 +4,30 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Layout
|
6
6
|
module EmptyLinesAroundBody
|
7
|
-
|
8
|
-
|
9
|
-
|
7
|
+
YAYOI_MSG_EXTRA = "ζ*'ヮ')ζ<うっうー!%<kind>sに空行が見つかりましたよー!".freeze
|
8
|
+
YAYOI_MSG_MISSING = "ζ*'ヮ')ζ<うっうー!%<kind>sのまわりに空行がないですよー!".freeze
|
9
|
+
YAYOI_MSG_DEFERRED = "ζ*'ヮ')ζ<うっうー!%<type>sの前に空行がないですよー!".freeze
|
10
|
+
|
11
|
+
def check_source(style, line_no, desc)
|
12
|
+
case style
|
13
|
+
when :no_empty_lines
|
14
|
+
check_line(style, line_no, message(YAYOI_MSG_EXTRA, desc), &:empty?)
|
15
|
+
when :empty_lines
|
16
|
+
check_line(style, line_no, message(YAYOI_MSG_MISSING, desc)) do |l|
|
17
|
+
!l.empty?
|
18
|
+
end
|
19
|
+
end
|
20
|
+
end
|
21
|
+
|
22
|
+
def deferred_message(node)
|
23
|
+
format(YAYOI_MSG_DEFERRED, type: node.type)
|
24
|
+
end
|
25
|
+
|
26
|
+
def message(type, desc)
|
27
|
+
kind = self.class::KIND
|
28
|
+
kind = self.class::YAYOI_KIND if kind.blank?
|
29
|
+
format(type, kind: kind, location: desc)
|
30
|
+
end
|
10
31
|
end
|
11
32
|
end
|
12
33
|
end
|
@@ -3,8 +3,19 @@
|
|
3
3
|
module RuboCop
|
4
4
|
module Cop
|
5
5
|
module EndKeywordAlignment
|
6
|
-
|
7
|
-
|
6
|
+
YAYOI_MSG = "ζ*'ヮ')ζ<うっうー! %<end_line>d, %<end_col>dにある`end`が" \
|
7
|
+
'%<align_line>d, %<align_col>d' \
|
8
|
+
'にある`%<source>s`とずれていますよー!'.freeze
|
9
|
+
|
10
|
+
def add_offense_for_misalignment(node, align_with)
|
11
|
+
end_loc = node.loc.end
|
12
|
+
msg = format(YAYOI_MSG, end_line: end_loc.line,
|
13
|
+
end_col: end_loc.column,
|
14
|
+
source: align_with.source,
|
15
|
+
align_line: align_with.line,
|
16
|
+
align_col: align_with.column)
|
17
|
+
add_offense(node, location: end_loc, message: msg)
|
18
|
+
end
|
8
19
|
end
|
9
20
|
end
|
10
21
|
end
|
@@ -0,0 +1,32 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
module RuboCop
|
4
|
+
module Cop
|
5
|
+
module MethodComplexity
|
6
|
+
def class_msg
|
7
|
+
msg = self.class::MSG
|
8
|
+
msg = self.class::YAYOI_MSG if msg.blank?
|
9
|
+
msg
|
10
|
+
end
|
11
|
+
|
12
|
+
def check_complexity(node, method_name)
|
13
|
+
# Accepts empty methods always.
|
14
|
+
return unless node.body
|
15
|
+
|
16
|
+
max = cop_config['Max']
|
17
|
+
complexity = complexity(node.body)
|
18
|
+
|
19
|
+
return unless complexity > max
|
20
|
+
|
21
|
+
msg = format(class_msg,
|
22
|
+
method: method_name,
|
23
|
+
complexity: complexity,
|
24
|
+
max: max)
|
25
|
+
|
26
|
+
add_offense(node, message: msg) do
|
27
|
+
self.max = complexity.ceil
|
28
|
+
end
|
29
|
+
end
|
30
|
+
end
|
31
|
+
end
|
32
|
+
end
|
@@ -3,7 +3,14 @@
|
|
3
3
|
module RuboCop
|
4
4
|
module Cop
|
5
5
|
module SpaceAfterPunctuation
|
6
|
-
|
6
|
+
YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!%<token>sの後ろにスペースがないですよー!".freeze
|
7
|
+
|
8
|
+
def investigate(processed_source)
|
9
|
+
each_missing_space(processed_source.tokens) do |token|
|
10
|
+
add_offense(token, location: token.pos,
|
11
|
+
message: format(YAYOI_MSG, token: kind(token)))
|
12
|
+
end
|
13
|
+
end
|
7
14
|
end
|
8
15
|
end
|
9
16
|
end
|
@@ -3,8 +3,8 @@
|
|
3
3
|
module RuboCop
|
4
4
|
module Cop
|
5
5
|
module UncommunicativeName
|
6
|
-
|
7
|
-
|
6
|
+
YAYOI_LENGTH_YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!" \
|
7
|
+
'%<name_type>sの長さは最低でも%<min>s文字以上にしましょうねー!'.freeze
|
8
8
|
|
9
9
|
def name_type(node)
|
10
10
|
@name_type ||= begin
|
@@ -14,6 +14,13 @@ module RuboCop
|
|
14
14
|
end
|
15
15
|
end
|
16
16
|
end
|
17
|
+
|
18
|
+
def length_offense(node, range)
|
19
|
+
add_offense(node, location: range,
|
20
|
+
message: format(YAYOI_LENGTH_YAYOI_MSG,
|
21
|
+
name_type: name_type(node).capitalize,
|
22
|
+
min: min_length))
|
23
|
+
end
|
17
24
|
end
|
18
25
|
end
|
19
26
|
end
|
@@ -4,9 +4,34 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Style
|
6
6
|
class CommentAnnotation < Cop
|
7
|
-
|
8
|
-
|
9
|
-
|
7
|
+
YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!`%<keyword>s`のような注釈は全部大きい文字にして" \
|
8
|
+
'コロン、スペースのあとにコメントを書きましょうねー!'.freeze
|
9
|
+
YAYOI_MISSING_NOTE = "ζ*'ヮ')ζ<うっうー!" \
|
10
|
+
'`%<keyword>s`があるけども、肝心のコメントがないですよー!'.freeze
|
11
|
+
|
12
|
+
def investigate(processed_source)
|
13
|
+
processed_source.comments.each_with_index do |comment, index|
|
14
|
+
next unless first_comment_line?(processed_source.comments, index) ||
|
15
|
+
inline_comment?(comment)
|
16
|
+
|
17
|
+
margin, first_word, colon, space, note = split_comment(comment)
|
18
|
+
next unless annotation?(comment) &&
|
19
|
+
!correct_annotation?(first_word, colon, space, note)
|
20
|
+
|
21
|
+
length = concat_length(first_word, colon, space)
|
22
|
+
add_offense(
|
23
|
+
comment,
|
24
|
+
location: annotation_range(comment, margin, length),
|
25
|
+
message: format(message(note), keyword: first_word)
|
26
|
+
)
|
27
|
+
end
|
28
|
+
end
|
29
|
+
|
30
|
+
private
|
31
|
+
|
32
|
+
def message(note)
|
33
|
+
note ? YAYOI_MSG : YAYOI_MISSING_NOTE
|
34
|
+
end
|
10
35
|
end
|
11
36
|
end
|
12
37
|
end
|
@@ -4,7 +4,18 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Style
|
6
6
|
class Documentation < Cop
|
7
|
-
|
7
|
+
YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!%<type>sのドキュメントのコメントを書きましょうねー".freeze
|
8
|
+
|
9
|
+
def check(node, body, type)
|
10
|
+
return if namespace?(body)
|
11
|
+
return if documentation_comment?(node) || nodoc_comment?(node)
|
12
|
+
return if compact_namespace?(node) &&
|
13
|
+
nodoc_comment?(outer_module(node).first)
|
14
|
+
|
15
|
+
add_offense(node,
|
16
|
+
location: :keyword,
|
17
|
+
message: format(YAYOI_MSG, type: type))
|
18
|
+
end
|
8
19
|
end
|
9
20
|
end
|
10
21
|
end
|
@@ -4,9 +4,13 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Style
|
6
6
|
class EmptyMethod < Cop
|
7
|
-
|
8
|
-
|
7
|
+
YAYOI_MSG_COMPACT = "ζ*'ヮ')ζ<うっうー!なにも書いてないメソッドは1行で書きましょうねー!".freeze
|
8
|
+
YAYOI_MSG_EXPANDED = "ζ*'ヮ')ζ<うっうー!なにも書いてないメソッドは1行で書かないで、" \
|
9
9
|
'`end`を次の行に書きましょうねー!'.freeze
|
10
|
+
|
11
|
+
def message(_node)
|
12
|
+
compact_style? ? YAYOI_MSG_COMPACT : YAYOI_MSG_EXPANDED
|
13
|
+
end
|
10
14
|
end
|
11
15
|
end
|
12
16
|
end
|
@@ -4,8 +4,17 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Style
|
6
6
|
class RedundantReturn < Cop
|
7
|
-
|
8
|
-
|
7
|
+
YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!冗長な`return`がありますよー!".freeze
|
8
|
+
YAYOI_MULTI_RETURN_YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!" \
|
9
|
+
'複数の値を返すときは配列を使いましょうねー!'.freeze
|
10
|
+
|
11
|
+
def message(node)
|
12
|
+
if !allow_multiple_return_values? && node.children.size > 1
|
13
|
+
"#{YAYOI_MSG} #{YAYOI_MULTI_RETURN_YAYOI_MSG}"
|
14
|
+
else
|
15
|
+
YAYOI_MSG
|
16
|
+
end
|
17
|
+
end
|
9
18
|
end
|
10
19
|
end
|
11
20
|
end
|
@@ -0,0 +1,21 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
module RuboCop
|
4
|
+
module Cop
|
5
|
+
module Style
|
6
|
+
class SymbolArray < Cop
|
7
|
+
PERCENT_YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!" \
|
8
|
+
'シンボルだけだけの配列には`%w`か`%W`を使いましょうねー!'.freeze
|
9
|
+
ARRAY_YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!配列には`[]`を使いましょうねー!".freeze
|
10
|
+
|
11
|
+
def message(_node)
|
12
|
+
if style == :percent
|
13
|
+
PERCENT_YAYOI_MSG
|
14
|
+
else
|
15
|
+
ARRAY_YAYOI_MSG
|
16
|
+
end
|
17
|
+
end
|
18
|
+
end
|
19
|
+
end
|
20
|
+
end
|
21
|
+
end
|
@@ -4,8 +4,16 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Style
|
6
6
|
class UnneededCondition < Cop
|
7
|
-
|
8
|
-
|
7
|
+
YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!ダブルパイプ(`||`)を使いましょうねー!".freeze
|
8
|
+
YAYOI_UNNEEDED_CONDITION = "ζ*'ヮ')ζ<うっうー!いらない条件がありますよー!".freeze
|
9
|
+
|
10
|
+
def message(node)
|
11
|
+
if node.modifier_form? || !node.else_branch
|
12
|
+
YAYOI_UNNEEDED_CONDITION
|
13
|
+
else
|
14
|
+
YAYOI_MSG
|
15
|
+
end
|
16
|
+
end
|
9
17
|
end
|
10
18
|
end
|
11
19
|
end
|
@@ -4,8 +4,17 @@ module RuboCop
|
|
4
4
|
module Cop
|
5
5
|
module Style
|
6
6
|
class WordArray < Cop
|
7
|
-
|
8
|
-
|
7
|
+
PERCENT_YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!" \
|
8
|
+
'文字列だけだけの配列には`%w`か`%W`を使いましょうねー!'.freeze
|
9
|
+
ARRAY_YAYOI_MSG = "ζ*'ヮ')ζ<うっうー!配列には`[]`を使いましょうねー!".freeze
|
10
|
+
|
11
|
+
def message(_node)
|
12
|
+
if style == :percent
|
13
|
+
PERCENT_YAYOI_MSG
|
14
|
+
else
|
15
|
+
ARRAY_YAYOI_MSG
|
16
|
+
end
|
17
|
+
end
|
9
18
|
end
|
10
19
|
end
|
11
20
|
end
|
data/lib/rubocop-yayoi.rb
CHANGED
@@ -1,5 +1,7 @@
|
|
1
1
|
require 'rubocop'
|
2
2
|
|
3
|
+
require_relative 'rubocop/cop/cop'
|
4
|
+
|
3
5
|
require_relative 'rubocop/formatter/progress_formatter'
|
4
6
|
require_relative 'rubocop/formatter/simple_text_formatter'
|
5
7
|
|
@@ -22,6 +24,7 @@ require_relative 'rubocop/cop/lint/useless_assignment'
|
|
22
24
|
|
23
25
|
require_relative 'rubocop/cop/mixin/empty_lines_around_body'
|
24
26
|
require_relative 'rubocop/cop/mixin/end_keyword_alignment'
|
27
|
+
require_relative 'rubocop/cop/mixin/method_complexity'
|
25
28
|
require_relative 'rubocop/cop/mixin/space_after_punctuation'
|
26
29
|
require_relative 'rubocop/cop/mixin/uncommunicative_name'
|
27
30
|
|
@@ -46,6 +49,7 @@ require_relative 'rubocop/cop/style/mutable_constant'
|
|
46
49
|
require_relative 'rubocop/cop/style/numeric_literals'
|
47
50
|
require_relative 'rubocop/cop/style/redundant_return'
|
48
51
|
require_relative 'rubocop/cop/style/string_literals'
|
52
|
+
require_relative 'rubocop/cop/style/symbol_array'
|
49
53
|
require_relative 'rubocop/cop/style/unneeded_condition'
|
50
54
|
require_relative 'rubocop/cop/style/unneeded_interpolation'
|
51
55
|
require_relative 'rubocop/cop/style/word_array'
|
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: rubocop-yayoi
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 0.0
|
4
|
+
version: 0.1.0
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- YutaGoto
|
8
8
|
autorequire:
|
9
9
|
bindir: exe
|
10
10
|
cert_chain: []
|
11
|
-
date: 2019-05-
|
11
|
+
date: 2019-05-21 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: rubocop
|
@@ -72,6 +72,7 @@ files:
|
|
72
72
|
- bin/setup
|
73
73
|
- lib/rubocop-yayoi.rb
|
74
74
|
- lib/rubocop/cop/.DS_Store
|
75
|
+
- lib/rubocop/cop/cop.rb
|
75
76
|
- lib/rubocop/cop/layout/def_end_alignment.rb
|
76
77
|
- lib/rubocop/cop/layout/empty_line_between_defs.rb
|
77
78
|
- lib/rubocop/cop/layout/empty_lines.rb
|
@@ -94,6 +95,7 @@ files:
|
|
94
95
|
- lib/rubocop/cop/mixin/.DS_Store
|
95
96
|
- lib/rubocop/cop/mixin/empty_lines_around_body.rb
|
96
97
|
- lib/rubocop/cop/mixin/end_keyword_alignment.rb
|
98
|
+
- lib/rubocop/cop/mixin/method_complexity.rb
|
97
99
|
- lib/rubocop/cop/mixin/space_after_punctuation.rb
|
98
100
|
- lib/rubocop/cop/mixin/uncommunicative_name.rb
|
99
101
|
- lib/rubocop/cop/naming/ascii_identifiers.rb
|
@@ -113,6 +115,7 @@ files:
|
|
113
115
|
- lib/rubocop/cop/style/numeric_literals.rb
|
114
116
|
- lib/rubocop/cop/style/redundant_return.rb
|
115
117
|
- lib/rubocop/cop/style/string_literals.rb
|
118
|
+
- lib/rubocop/cop/style/symbol_array.rb
|
116
119
|
- lib/rubocop/cop/style/unneeded_condition.rb
|
117
120
|
- lib/rubocop/cop/style/unneeded_interpolation.rb
|
118
121
|
- lib/rubocop/cop/style/word_array.rb
|