Lint

Author: TylerZeroMaster

lib/kitchen/directions/bake_toc.rb

@@ -9,11 +9,11 @@ def self.v1(book:, options: { cases: false })
           numbering_options: { mode: :chapter_page, separator: '.' },
           controller: {
             get_chapter_toc_title: nil,
-            get_unit_toc_title: nil,
+            get_unit_toc_title: nil
           }
         )
         controller = options[:controller]
-        @toc_title_for_unit = 
+        @toc_title_for_unit =
           if controller[:get_unit_toc_title].nil?
             lambda do |unit|
               number = unit.os_number(options[:numbering_options])
@@ -26,8 +26,8 @@ def self.v1(book:, options: { cases: false })
           else
             controller[:get_unit_toc_title]
           end
-        
-        @toc_title_for_chapter = 
+
+        @toc_title_for_chapter =
           if controller[:get_chapter_toc_title].nil?
             lambda do |chapter|
               number = chapter.os_number(options[:numbering_options])
@@ -107,7 +107,7 @@ def self.li_for_chapter(chapter)
             li_for_composite_chapter(child)
           end
         end.join("\n")
-        
+
         <<~HTML
           <li class="os-toc-chapter" cnx-archive-shortid="" cnx-archive-uri="" data-toc-type="chapter">
             <a href="##{chapter.title.id}">

lib/kitchen/element_base.rb

@@ -402,7 +402,9 @@ def number_parts(mode, unit_offset:, chapter_offset:, page_offset:)
         else
           unit = ancestor(:unit)
           chapter = ancestor(:chapter)
-          [unit.count_in(:book) + unit_offset, chapter.count_in(:unit) + chapter_offset, count_in(:chapter) + page_offset]
+          [unit.count_in(:book) + unit_offset,
+           chapter.count_in(:unit) + chapter_offset,
+           count_in(:chapter) + page_offset]
         end
       else
         # Further levels of nesting are not currently supported because of how
@@ -424,13 +426,13 @@ def os_number(options={})
         separator: '.',
         unit_offset: 0,
         chapter_offset: 0,
-        page_offset: 0,
+        page_offset: 0
       )
       parts = number_parts(
         options[:mode],
         unit_offset: options[:unit_offset],
         chapter_offset: options[:chapter_offset],
-        page_offset: options[:page_offset],
+        page_offset: options[:page_offset]
       )
       parts.join(options[:separator])
     end

lib/recipes/algebra-1/recipe.rb

@@ -22,12 +22,10 @@
       unit.chapters.each_with_index do |ch, ch_idx|
         title = ch.title.to_s
         is_unnumbered_chapter = (
-          ch_idx == 0 or
+          ch_idx.zero? or
           title.include?('Overview and Readiness') or
           title.include?('Project'))
-        if is_unnumbered_chapter
-          ch[unnumbered_chapter_marker] = 'true'
-        end
+        ch[unnumbered_chapter_marker] = 'true' if is_unnumbered_chapter
       end
     end
   end
@@ -67,7 +65,7 @@
     numbered: { mode: :unit_chapter_page, page_offset: -1 }
   }
   chapters_by_type = {
-    unnumbered: lambda { book.chapters("$[#{unnumbered_chapter_marker}]") },
+    unnumbered: -> { book.chapters("$[#{unnumbered_chapter_marker}]") },
     numbered: lambda {
       book
         .units("$:not([#{unnumbered_unit_marker}])")
@@ -188,7 +186,7 @@
     book.pages('$.unit-closer').to_a
   ).each do |element|
     element.title.replace_children(
-      with: %%<span data-type="" itemprop="" class="os-text">#{element.title_text}</span>%
+      with: %(<span data-type="" itemprop="" class="os-text">#{element.title_text}</span>)
     )
   end
 
@@ -199,7 +197,7 @@
       controller: {
         get_unit_toc_title: lambda do |unit|
           if unit[unnumbered_unit_marker]
-            (%%<span data-type="" itemprop="" class="os-text">#{unit.title_text}</span>%)
+            %(<span data-type="" itemprop="" class="os-text">#{unit.title_text}</span>)
           else
             number = unit.os_number({ mode: :unit_chapter_page,
                                       unit_offset: -skipped_units })