lib/todoist/plugin.rb in danger-todoist-0.3.0 vs lib/todoist/plugin.rb in danger-todoist-1.0.0

@@ -65,14 +65,13 @@
       find_todos if @todos.nil?
       return if @todos.empty?
 
       markdown("#### Todos left in files")
 
-      @todos.each do |todo|
-        text = ": #{todo.text}" if todo.text
-        markdown("- #{todo.file}#{text}")
-      end
+      @todos
+        .group_by(&:file)
+        .each { |file, todos| print_todos_per_file(file, todos) }
     end
 
     #
     # Returns the list of todos in the current diff set
     #
@@ -82,9 +81,16 @@
       find_todos if @todos.nil?
       @todos
     end
 
     private
+
+    def print_todos_per_file(file, todos)
+      markdown("- #{file}")
+      todos
+        .select(&:text)
+        .each { |todo| markdown("  - #{todo.text}") }
+    end
 
     def call_method_for_todos(method)
       find_todos if @todos.nil?
       public_send(method, message, sticky: false) unless @todos.empty?
     end