exercism
Prism syntax highlighting broken for Smalltalk empty strings
If you look at the rendered test suite pane for ReverseString, the colourised code is broken due to a quoted apostrophe. e.g. 'don''t break'
I'm not sure where the colouriser came from (I think @bencoman knows) - but we should report this issue and get it fixed.
@bencoman - the following is from the Test Suite tab of my Pharo track for the ReserseString exercise:
So you mean its on the Exercism web site?
I can't find it - but maybe that is because I'm in "independent mode"
It looks like Exercism used to use Rouge syntax highlighter until Aug 21, 2018... https://github.com/exercism/website/commit/464287fa3#diff-8b7db4d5
and now use Prism... https://github.com/exercism/exercism/issues/4044
Can reproduce the above snapshot online at... https://prismjs.com/test.html pasting in code from... https://github.com/exercism/pharo/blob/master/dev/src/Exercism/ReverseStringTest.class.st
Concise test...
anObject withEmptyString: '' shouldNotBeGreenHere: 'shouldBeGreenHere'
https://prismjs.com/test.html displays incorrect...
http://rouge.jneen.net/ displays correct...
Actually I discover the error starts on line 34 of the snapshot due to the empty string. Reported here... https://github.com/PrismJS/prism/issues/1746
Confirming https://prismjs.com/test.html displays quoted apostrophes correctly...
Ben managed to get a fix in prism (https://github.com/PrismJS/prism/pull/1747) but not sure how exercism pulls in prism? When there is the next release, will it just get included?