Conversation
|
Cucumber and Minitest pass on my machine. |
|
Hm, tests seem to fail due to 1 rdiscount failure. Maybe it's related to the i18n version? Hm. Also, I can't believe we still test rdiscount – we don't waste our energy on rdiscount in Jekyll 4, do we? Failure:
TestRdiscount#test_: rdiscount should render toc. [/home/runner/work/jekyll/jekyll/test/test_rdiscount.rb:47]
Minitest::Assertion:
--- expected
+++ actual
@@ -1,13 +1,13 @@
-"<a name=\"Header.1\"></a>
+"<a name=\"Header-1\"></a>
<h1>Header 1</h1>
-<a name=\"Header.2\"></a>
+<a name=\"Header-2\"></a>
<h2>Header 2</h2>
<p><ul>
- <li><a href=\"#Header.1\">Header 1</a>
+ <li><a href=\"#Header-1\">Header 1</a>
<ul>
- <li><a href=\"#Header.2\">Header 2</a></li>
+ <li><a href=\"#Header-2\">Header 2</a></li>
</ul>
</li>
</ul> |
|
Rdiscount failure also occurred here: #9272 So it must be a breaking change. We should just update our test. |
gem I recommend updating the test (or locking to last known compatible version) via a separate pull request (or a direct commit to the
Jekyll 4 dropped built-in support for |
ec41d1b to
5dcb2c8
Compare
5dcb2c8 to
2325123
Compare
This is a 🙋 feature or enhancement.
The test suite passes locally (run
script/cibuildto verify this)Summary
The
i18nplugin was added in 93e3eb06d2 to allow for some extra options to theslugifyfilter. At the time, we locked it to i18n v0.x, not allowing v1.0. This is a reasonable solution since we didn't know what v1 would hold. v1 is out (and has been out a while) and it's now required by the newest version of activesupport, a common library used by/with Jekyll. We should update v3.x to allow compatibility with this newer gem version.Context
Closes #9268.
/cc github/pages-gem#866