iv-org-mirror/shard-ameba
1
0
Fork 0
mirror of https://gitea.invidious.io/iv-org/shard-ameba.git synced 2024-08-15 00:53:29 +00:00
Code Issues Projects Releases Packages Wiki Activity

Several refactors in Style/GuardClause rule

Browse source 
Mostly readability related
This commit is contained in:
Sijawusz Pur Rahnama 2022-11-14 01:06:09 +01:00
parent 353c0a1583
commit ed6085e6a6
1 changed files with 6 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/ameba/rule/style/guard_clause.cr
View file

@ -75,8 +75,8 @@ module Ameba::Rule::Style
end
case final_expression
when Crystal::If then check_ending_if(source, final_expression)
when Crystal::Unless then check_ending_if(source, final_expression)
when Crystal::If, Crystal::Unless
check_ending_if(source, final_expression)
end
end
@ -92,7 +92,8 @@ module Ameba::Rule::Style
return unless guard_clause && parent && conditional_keyword
report_issue(source, node, guard_clause_source(source, guard_clause, parent), conditional_keyword)
guard_clause_source = guard_clause_source(source, guard_clause, parent)
report_issue(source, node, guard_clause_source, conditional_keyword)
end
private def check_ending_if(source, node)
@ -116,7 +117,8 @@ module Ameba::Rule::Style
end_loc = end_end_loc.adjust(column_number: {{1 - "end".size}})
issue_for keyword_loc, keyword_end_loc, MSG % example do |corrector|
corrector.replace(keyword_loc, keyword_end_loc, "#{scope_exiting_keyword} #{conditional_keyword}")
replacement = "#{scope_exiting_keyword} #{conditional_keyword}"
corrector.replace(keyword_loc, keyword_end_loc, replacement)
corrector.remove(end_loc, end_end_loc)
end
else