github-markup 4.0.1 → 5.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 460b133e720fe08106c84643e1f7b1ca9e2878fbbd3c6cf5865a287f5e5bf92f
-  data.tar.gz: 16a8c4145ec42146e945c7cd4cfb3e7819701f099e5de43528b770087ce114b9
+  metadata.gz: 0c0a1b208f2db4d8dbe2da7b892e9ab357901f60a4474967ffce14103a57cbea
+  data.tar.gz: a4d1eb586fdcf139731eb53550af3ce1a6bd4216dc44a2b7bd3bb6f69f237ecf
 SHA512:
-  metadata.gz: 7c6307defe17fe273441c24a615b15840fdfd7a36e2e1585ef7ce84c6bcbf39c6f512a1a354541e713bb957f1f42771423fe06f354f37fe6cd1f4881357980dc
-  data.tar.gz: 57fd400b0a38e898eb27da5db1d0335eea594530cce97b5748bb70d6d8594d79e7ec90a72dbfe9e16e85957b93d3da42d7e2805dd57eec28f10dd9afb6153428
+  metadata.gz: 70544a9808c1858e816e37e283ecc2688b9c775b642f609f3cf92eefb4b6c6cdcf91386c1f058c5e13c90afd537aa99caf188f4ed8a78cd657f6ec50159f8d6f
+  data.tar.gz: a101a59c827e06dd5f687a22c5f7fdfe0316d4ff2cfee7d36d13df0b2bfd65669f1340a072013a02e9e120705d31d0bfaf5efe42b6024e127005b6541bb70dce

data/.github/workflows/ci.yml

@@ -11,10 +11,10 @@ jobs:
     strategy:
       matrix:
         ruby:
-          - "2.4"
-          - "2.5"
-          - "2.6"
-          - "2.7"
+          - "3.0"
+          - "3.1"
+          - "3.2"
+          - "3.3"
       fail-fast: false
 
     steps:

data/.gitignore

@@ -4,4 +4,5 @@ pkg/
 Gemfile.lock
 .project
 .buildpath
-*~ 
+*~
+vendor/

data/CONTRIBUTING.md

@@ -11,7 +11,7 @@ This library's only job is to decide which markup format to use and call out to
 If you are having an issue with:
 
 * **Syntax highlighting** - see [github/linguist](https://github.com/github/linguist/blob/master/CONTRIBUTING.md#fixing-syntax-highlighting)
-* **Markdown on GitHub** - contact support@github.com
+* **Markdown on GitHub** - contact [GitHub Support](https://support.github.com/)
 * **Styling issues on GitHub** - see [primer-markdown](https://github.com/primer/primer-css/tree/master/modules/primer-markdown) module in the [primer/primer-css](https://github.com/primer/primer-css) repository
 
 Anything else - [search open issues](https://github.com/github/markup/issues) or create an issue and and we'll help point you in the right direction.

data/Gemfile

@@ -1,17 +1,17 @@
 source "http://rubygems.org"
 gemspec
 
-gem "posix-spawn", :platforms => :ruby
 gem "redcarpet", :platforms => :ruby
 gem "kramdown", :platforms => :jruby
 gem "RedCloth"
 # using a tag version here because 0.18.3 was not published by the author to encourage users to upgrade.
 # however we want to bump up to this version since this has a security patch
 gem "commonmarker", git: "https://github.com/gjtorikian/commonmarker.git", tag: "v0.18.3"
-gem "rdoc", "~>3.6"
+gem "rdoc", "~> 6.7.0"
 gem "org-ruby", "= 0.9.9"
 gem "creole", "~>0.3.6"
 gem "wikicloth", "=0.8.3"
 gem "twitter-text", "~> 1.14"
 gem "asciidoctor", "~> 2.0.5"
 gem "rake"
+gem "rexml"

data/HISTORY.md

@@ -1,3 +1,13 @@
+## 5.0.0 - 2024-06-17
+* Drop support for Ruby versions < 3
+* Bump nokogiri from 1.8.1 to 1.16.5
+* Bump nokogiri-diff from 0.2.0 to 0.3.0
+* Bump rdoc from 3.6 to 6.7.0
+* Update CommandImplementation to better support large files (affecting RST and POD6 rendering)
+
+## 4.0.2 - 2023-10-10
+* Add support for .mdx files in markdown
+
 ## 4.0.1 - 2022-03-07
 * Update to commonmarker 0.18.3; There isn't a version on RubyGems for this, so this is pointing to a [tag version on GitHub](https://github.com/gjtorikian/commonmarker/blob/v0.18.3/commonmarker.gemspec)
 

data/github-markup.gemspec

@@ -13,8 +13,9 @@ Gem::Specification.new do |s|
   s.homepage      = "https://github.com/github/markup"
   s.license       = "MIT"
 
+  s.required_ruby_version = '>= 3.0.0'
+
   s.files         = `git ls-files`.split($\)
-  s.files        += Dir['vendor/**/*']
   s.executables   = s.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
   s.test_files    = s.files.grep(%r{^(test|spec|features)/})
   s.require_paths = %w[lib]
@@ -24,7 +25,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'minitest', '~> 5.4', '>= 5.4.3'
   s.add_development_dependency 'html-pipeline', '~> 1.0'
   s.add_development_dependency 'sanitize', '>= 4.6.3'
-  s.add_development_dependency 'nokogiri', '~> 1.8.1'
-  s.add_development_dependency 'nokogiri-diff', '~> 0.2.0'
+  s.add_development_dependency 'nokogiri', '~> 1.16.5'
+  s.add_development_dependency 'nokogiri-diff', '~> 0.3.0'
   s.add_development_dependency "github-linguist", ">= 7.1.3"
 end

data/lib/github/markup/command_implementation.rb

@@ -1,9 +1,4 @@
-begin
-  require "posix-spawn"
-rescue LoadError
-  require "open3"
-end
-
+require "open3"
 require "github/markup/implementation"
 
 
@@ -39,28 +34,13 @@ module GitHub
         end
       end
 
-      if defined?(POSIX::Spawn)
-        def execute(command, target)
-          spawn = POSIX::Spawn::Child.new(*command, :input => target)
-          if spawn.status.success?
-            sanitize(spawn.out, target.encoding)
-          else
-            raise CommandError.new(spawn.err.strip)
-          end
-        end
-      else
-        def execute(command, target)
-          output = Open3.popen3(*command) do |stdin, stdout, stderr, wait_thr|
-            stdin.puts target
-            stdin.close
-            if wait_thr.value.success?
-              stdout.readlines
-            else
-              raise CommandError.new(stderr.readlines.join('').strip)
-            end
-          end
-          sanitize(output.join(''), target.encoding)
-        end
+      def execute(command, target)
+        # capture3 blocks until both buffers are written to and the process terminates, but
+        # it won't allow either buffer to fill up
+        stdout, stderr, status = Open3.capture3(*command, stdin_data: target)
+
+        raise CommandError.new(stderr) unless status.success?
+        sanitize(stdout, target.encoding)
       end
 
       def sanitize(input, encoding)

data/lib/github/markup/markdown.rb

@@ -31,8 +31,8 @@ module GitHub
 
       def initialize
         super(
-          /md|mkdn?|mdwn|mdown|markdown|litcoffee/i,
-          ["Markdown", "Literate CoffeeScript"])
+          /md|mkdn?|mdwn|mdown|markdown|mdx|litcoffee/i,
+          ["Markdown", "MDX", "Literate CoffeeScript"])
       end
 
       def load

data/lib/github-markup.rb

@@ -1,6 +1,6 @@
 module GitHub
   module Markup
-    VERSION = '4.0.1'
+    VERSION = '5.0.0'
     Version = VERSION
   end
 end

data/test/markup_test.rb

@@ -66,6 +66,11 @@ class MarkupTest < Minitest::Test
         f.close_write
         f.read
       end
+
+      if ENV['UPDATE']
+        File.open(expected_file, 'w') { |f| f.write actual }
+      end
+
       assert_html_equal expected, actual, <<message
 #{File.basename expected_file}'s contents are not html equal to output:
 #{diff}
@@ -106,7 +111,7 @@ message
     begin
       GitHub::Markup.render('README.java', "stop swallowing errors", symlink: false)
     rescue GitHub::Markup::CommandError => e
-      assert_equal "failure message", e.message
+      assert_equal "failure message", e.message.strip
     else
       fail "an exception was expected but was not raised"
     end

data/test/markups/README.asciidoc.html

@@ -59,4 +59,4 @@ Here is some source code.
 <p>content</p>
 </div>
 </div>
-</div>
+</div>

data/test/markups/README.hidetitle.asciidoc.html

@@ -1,3 +1,3 @@
 <div>
 <p>This test verifies the author can disable the document title by adding <code>:!showtitle:</code> to the document header.</p>
-</div>
+</div>

data/test/markups/README.litcoffee.html

@@ -52,4 +52,4 @@ like links and unordered lists, are fine:</p>
 </ul>
 <p>Tabs work too:</p>
 <p>test "tabbed code", -&gt;
-ok yes</p>
+ok yes</p>