Highlight using Pygments rather than Geshi
GeSHi is unmaintained, lacks support for many popular modern languages, and
suffers from deep architectural flaws, chief among them the inconsistent
tokenization of different languages, each of which requires a custom
stylesheet.
Pygments is a well-maintained alternative. It is, by my count, the most popular
syntax highlighting library around. It is BSD-licensed, actively maintained,
and is widely used in PHP projects.
To keep this easy to review, this change does not include update for l10n
files, and it does not delete the geshi/ directory. I will do those in a
separate patch.
The chief change between this and the previous implementation is that errors
result in the code block not being highlighted, as opposed to not being printed
at all, having been replaced by an angry red error message. I think that is the
right user experience. If you go to StackOverflow or GitHub and try to mark up
your code block as being written in some language that their highlighter
doesn't know about, you don't get an error message -- the code simply doesn't
get highlighted.
Because we don't recursively load dependencies for extensions, to test this,
you will need to create a composer.local.json in $IP and add:
{
"extra": {
"merge-plugin": {
"include": [
"extensions/SyntaxHighlight_GeSHi/composer.json"
]
}
}
}
Then run `composer update`.
Bug: T85794
Change-Id: I07446ec9893fae3d1e394f435d3d95cf8be6bc33
2015-06-16 03:52:58 +00:00
|
|
|
{
|
2017-05-20 15:58:14 +00:00
|
|
|
"name": "mediawiki/syntax-highlight",
|
2015-10-23 22:33:06 +00:00
|
|
|
"description": "Syntax highlighting extension for MediaWiki",
|
|
|
|
|
"require": {
|
2017-04-04 19:59:34 +00:00
|
|
|
"symfony/process": "~3.2"
|
2015-10-23 22:33:06 +00:00
|
|
|
},
|
|
|
|
|
"require-dev": {
|
2016-01-05 18:41:02 +00:00
|
|
|
"jakub-onderka/php-parallel-lint": "0.9.2",
|
2017-07-08 09:51:33 +00:00
|
|
|
"mediawiki/mediawiki-codesniffer": "0.10.0",
|
2017-05-05 19:40:02 +00:00
|
|
|
"jakub-onderka/php-console-highlighter": "0.3.2"
|
2015-10-23 22:33:06 +00:00
|
|
|
},
|
2015-12-02 21:21:11 +00:00
|
|
|
"license": "GPL-2.0+",
|
2015-10-23 22:33:06 +00:00
|
|
|
"authors": [
|
|
|
|
|
{
|
|
|
|
|
"name": "Brion Vibber",
|
|
|
|
|
"email": "brion@wikimedia.org"
|
|
|
|
|
},
|
|
|
|
|
{
|
|
|
|
|
"name": "Niklas Laxström",
|
|
|
|
|
"email": "nlaxstrom@wikimedia.org"
|
|
|
|
|
},
|
|
|
|
|
{
|
|
|
|
|
"name": "Ori Livneh",
|
|
|
|
|
"email": "ori@wikimedia.org"
|
|
|
|
|
},
|
|
|
|
|
{
|
|
|
|
|
"name": "Rob Church"
|
|
|
|
|
},
|
|
|
|
|
{
|
|
|
|
|
"name": "Tim Starling",
|
|
|
|
|
"email": "tstarling@wikimedia.org"
|
|
|
|
|
}
|
|
|
|
|
],
|
|
|
|
|
"scripts": {
|
2017-04-28 21:57:37 +00:00
|
|
|
"fix": "phpcbf",
|
2015-10-23 22:33:06 +00:00
|
|
|
"test": [
|
2017-06-13 10:20:31 +00:00
|
|
|
"parallel-lint . --exclude vendor --exclude node_modules",
|
2015-10-23 22:33:06 +00:00
|
|
|
"phpcs -p -s"
|
|
|
|
|
]
|
|
|
|
|
}
|
Highlight using Pygments rather than Geshi
GeSHi is unmaintained, lacks support for many popular modern languages, and
suffers from deep architectural flaws, chief among them the inconsistent
tokenization of different languages, each of which requires a custom
stylesheet.
Pygments is a well-maintained alternative. It is, by my count, the most popular
syntax highlighting library around. It is BSD-licensed, actively maintained,
and is widely used in PHP projects.
To keep this easy to review, this change does not include update for l10n
files, and it does not delete the geshi/ directory. I will do those in a
separate patch.
The chief change between this and the previous implementation is that errors
result in the code block not being highlighted, as opposed to not being printed
at all, having been replaced by an angry red error message. I think that is the
right user experience. If you go to StackOverflow or GitHub and try to mark up
your code block as being written in some language that their highlighter
doesn't know about, you don't get an error message -- the code simply doesn't
get highlighted.
Because we don't recursively load dependencies for extensions, to test this,
you will need to create a composer.local.json in $IP and add:
{
"extra": {
"merge-plugin": {
"include": [
"extensions/SyntaxHighlight_GeSHi/composer.json"
]
}
}
}
Then run `composer update`.
Bug: T85794
Change-Id: I07446ec9893fae3d1e394f435d3d95cf8be6bc33
2015-06-16 03:52:58 +00:00
|
|
|
}
|