Skip to content

A
annotate
Commit 3333c359 by Shu Fujita Committed by Andrew W. Lee
Options

Refactor AnnotateRoutes.annotate_routes and .rewrite_contents_with_header (#729)

I refactored `AnnotateRoutes.annotate_routes` and `.rewrite_contents_with_header` in order to add an empty string to `new_content` array in `.annotate_routes`. I will refactor `.rewrite_contents_with_header` more, and remove `. rewrite_contents_with_header` in the next PR.
parent 13eeb969
Showing with 3 additions and 3 deletions
lib/annotate/annotate_routes.rb
...@@ -65,9 +65,6 @@ module AnnotateRoutes ...@@ -65,9 +65,6 @@ module AnnotateRoutes
def rewrite_contents_with_header(existing_text, header, options = {}) def rewrite_contents_with_header(existing_text, header, options = {})
content, header_position = strip_annotations(existing_text) content, header_position = strip_annotations(existing_text)
new_content = annotate_routes(header, content, header_position, options) new_content = annotate_routes(header, content, header_position, options)
# Make sure we end on a trailing newline.
new_content << '' unless new_content.last == ''
new_text = new_content.join("\n") new_text = new_content.join("\n")
if existing_text == new_text if existing_text == new_text
...@@ -195,6 +192,9 @@ module AnnotateRoutes ...@@ -195,6 +192,9 @@ module AnnotateRoutes
new_content = magic_comments_map + content + header new_content = magic_comments_map + content + header
end end
# Make sure we end on a trailing newline.
new_content << '' unless new_content.last == ''
new_content new_content
end end
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment