Allow trailing pipes to be ommitted

ruby · Oct 6, 2022 · 1318048 · 1318048
1 parent 993f253
commit 1318048
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/lib/rdoc/markdown.kpeg b/lib/rdoc/markdown.kpeg
@@ -1201,12 +1201,12 @@ Table = &{ github? }
         TableRow:header TableLine:line TableRow+:body
         { table = RDoc::Markup::Table.new(header, line, body) }
 
-TableRow = TableItem+:row "|" @Newline
+TableRow = TableItem+:row "|"? @Newline
            { row }
 TableItem = "|" < (!"|" !@Newline .)+ >
             { text.strip }
 
-TableLine = TableColumn+:line "|" @Newline
+TableLine = TableColumn+:line "|"? @Newline
             { line }
 TableColumn = "|" < ( "-"+ ":"? | ":" "-"* ) >
               { text.start_with?(":") ? :left :

diff --git a/test/rdoc/test_rdoc_markdown.rb b/test/rdoc/test_rdoc_markdown.rb
@@ -1062,6 +1062,25 @@ def test_gfm_table
     assert_equal expected, doc
   end
 
+  def test_gfm_table_2
+    doc = parse <<~MD
+    | Cmd | Returns | Meaning
+    |-----|---------|--------
+    | "b" | boolean | True if file1 is a block device
+    | "c" | boolean | True if file1 is a character device
+    MD
+
+    head = %w[Cmd Returns Meaning]
+    align = [nil, nil, nil]
+    body = [
+      ['"b"', 'boolean', 'True if file1 is a block device'],
+      ['"c"', 'boolean', 'True if file1 is a character device'],
+    ]
+    expected = doc(@RM::Table.new(head, align, body))
+
+    assert_equal expected, doc
+  end
+
   def parse text
     @parser.parse text
   end