Skip to content

Commit 5d7116f

Browse files
authored
Merge pull request #597 from crystal-ameba/revert-595-autocorrect-multiline-curly-block
Revert "Support autocorrection in `Style/MultilineCurlyBlock`"
2 parents 2afc23e + 2ea577f commit 5d7116f

File tree

2 files changed

+2
-11
lines changed

2 files changed

+2
-11
lines changed

spec/ameba/rule/style/multiline_curly_block_spec.cr

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -25,18 +25,12 @@ module Ameba::Rule::Style
2525
end
2626

2727
it "reports if there is a multi-line curly block" do
28-
source = expect_issue subject, <<-CRYSTAL
28+
expect_issue subject, <<-CRYSTAL
2929
foo {
3030
# ^ error: Use `do`...`end` instead of curly brackets for multi-line blocks
3131
:bar
3232
}
3333
CRYSTAL
34-
35-
expect_correction source, <<-CRYSTAL
36-
foo do
37-
:bar
38-
end
39-
CRYSTAL
4034
end
4135
end
4236
end

src/ameba/rule/style/multiline_curly_block.cr

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,7 @@ module Ameba::Rule::Style
4040
return if start_location.line_number == end_location.line_number
4141
return unless source.code[source.pos(start_location)]? == '{'
4242

43-
issue_for node, MSG do |corrector|
44-
corrector.replace(start_location, start_location, "do")
45-
corrector.replace(end_location, end_location, "end")
46-
end
43+
issue_for node, MSG
4744
end
4845
end
4946
end

0 commit comments

Comments
 (0)