Merge branch 'master' into regex-test-coverage

.eslintrc.js

@@ -8,6 +8,9 @@ module.exports = {
 
 		// stylistic rules
 		'brace-style': ['warn', '1tbs', { allowSingleLine: true }],
+		'curly': ['warn', 'all'],
+		'eol-last': 'warn',
+		'no-multiple-empty-lines': ['warn', { max: 2, maxBOF: 0, maxEOF: 0 }],
 		'no-tabs': ['warn', { allowIndentationTabs: true }],
 		'no-var': 'error',
 		'one-var': ['warn', 'never'],

README.md

@@ -17,6 +17,7 @@ Prism depends on community contributions to expand and cover a wider array of us
 
 - Read the [documentation](https://prismjs.com/extending.html). Prism was designed to be extensible.
 - Do not edit `prism.js`, it’s just the version of Prism used by the Prism website and is built automatically. Limit your changes to the unminified files in the `components/` folder. `prism.js` and all minified files are also generated automatically by our build system.
+- Use `npm ci` to install Prism's dependencies. Do not use `npm install` because it will cause non-deterministic builds.
 - The build system uses [gulp](https://github.com/gulpjs/gulp) to minify the files and build `prism.js`. With all of Prism's dependencies installed, you just need to run the command `npm run build`.
 - Please follow the code conventions used in the files already. For example, I use [tabs for indentation and spaces for alignment](http://lea.verou.me/2012/01/why-tabs-are-clearly-superior/). Opening braces are on the same line, closing braces on their own line regardless of construct. There is a space before the opening brace. etc etc.
 - Please try to err towards more smaller PRs rather than a few huge PRs. If a PR includes changes that I want to merge and also changes that I don't, handling it becomes difficult.

assets/code.js

@@ -119,7 +119,9 @@
 
 // calc()
 (function () {
-	if (!window.PrefixFree) return;
+	if (!window.PrefixFree) {
+		return;
+	}
 
 	if (PrefixFree.functions.indexOf('calc') == -1) {
 		var style = document.createElement('_').style;

assets/download.js

@@ -187,13 +187,16 @@
 			}
 
 			function getLanguageTitle(lang) {
-				if (!lang.aliasTitles)
+				if (!lang.aliasTitles) {
 					return lang.title;
+				}
 
 				var titles = [lang.title];
-				for (var alias in lang.aliasTitles)
-					if (lang.aliasTitles.hasOwnProperty(alias))
+				for (var alias in lang.aliasTitles) {
+					if (lang.aliasTitles.hasOwnProperty(alias)) {
 						titles.push(lang.aliasTitles[alias]);
+					}
+				}
 				return titles.join(' + ');
 			}
 

components/prism-ada.js

@@ -16,4 +16,4 @@ Prism.languages.ada = {
 	'punctuation': /\.\.?|[,;():]/,
 	'char': /'.'/,
 	'variable': /\b[a-z](?:[_a-z\d])*\b/i
-};
+};

components/prism-arff.js

@@ -7,4 +7,4 @@ Prism.languages.arff = {
 	'keyword': /@(?:attribute|data|end|relation)\b/i,
 	'number': /\b\d+(?:\.\d+)?\b/,
 	'punctuation': /[{},]/
-};
+};

components/prism-bbcode.js

@@ -26,4 +26,4 @@ Prism.languages.bbcode = {
 	}
 };
 
-Prism.languages.shortcode = Prism.languages.bbcode;
+Prism.languages.shortcode = Prism.languages.bbcode;

components/prism-brainfuck.js

@@ -17,4 +17,4 @@ Prism.languages.brainfuck = {
 	},
 	'operator': /[.,]/,
 	'comment': /\S+/
-};
+};

components/prism-csp.js

@@ -26,4 +26,4 @@ Prism.languages.csp = {
 		pattern: /(?:'unsafe-(?:allow-redirects|dynamic|eval|hash-attributes|hashed-attributes|hashes|inline)'|\*)/i,
 		alias: 'function'
 	}
-};
+};

components/prism-dot.js

@@ -74,4 +74,3 @@
 	Prism.languages.gv = Prism.languages.dot;
 
 }(Prism));
-

components/prism-haxe.js

@@ -42,4 +42,4 @@ Prism.languages.insertBefore('haxe', 'keyword', {
 	}
 });
 Prism.languages.haxe['string'].inside['interpolation'].inside.rest = Prism.languages.haxe;
-delete Prism.languages.haxe['class-name'];
+delete Prism.languages.haxe['class-name'];

components/prism-icon.js

@@ -17,4 +17,4 @@ Prism.languages.icon = {
 	'function': /(?!\d)\w+(?=\s*[({]|\s*!\s*\[)/,
 	'operator': /[+-]:(?!=)|(?:[\/?@^%&]|\+\+?|--?|==?=?|~==?=?|\*\*?|\|\|\|?|<(?:->?|<?=?)|>>?=?)(?::=)?|:(?:=:?)?|[!.\\|~]/,
 	'punctuation': /[\[\](){},;]/
-};
+};

components/prism-javascript.js

@@ -26,7 +26,7 @@ Prism.languages.javascript['class-name'][0].pattern = /(\b(?:class|interface|ext
 
 Prism.languages.insertBefore('javascript', 'keyword', {
 	'regex': {
-		pattern: /((?:^|[^$\w\xA0-\uFFFF."'\])\s]|\b(?:return|yield))\s*)\/(?:\[(?:[^\]\\\r\n]|\\.)*]|\\.|[^/\\\[\r\n])+\/[gimyus]{0,6}(?=(?:\s|\/\*(?:[^*]|\*(?!\/))*\*\/)*(?:$|[\r\n,.;:})\]]|\/\/))/,
+		pattern: /((?:^|[^$\w\xA0-\uFFFF."'\])\s]|\b(?:return|yield))\s*)\/(?:\[(?:[^\]\\\r\n]|\\.)*]|\\.|[^/\\\[\r\n])+\/[dgimyus]{0,7}(?=(?:\s|\/\*(?:[^*]|\*(?!\/))*\*\/)*(?:$|[\r\n,.;:})\]]|\/\/))/,
 		lookbehind: true,
 		greedy: true,
 		inside: {

components/prism-keyman.js

@@ -11,4 +11,4 @@ Prism.languages.keyman = {
 	'number': /\b(?:U\+[\dA-F]+|d\d+|x[\da-f]+|\d+)\b/i, // U+####, x###, d### characters and numbers
 	'operator': /[+>\\,()]/,
 	'tag': /\$(?:keyman|kmfl|weaver|keymanweb|keymanonly):/i   // prefixes
-};
+};

components/prism-markup-templating.js

@@ -39,8 +39,9 @@
 					var placeholder;
 
 					// Check for existing strings
-					while (env.code.indexOf(placeholder = getPlaceholder(language, i)) !== -1)
+					while (env.code.indexOf(placeholder = getPlaceholder(language, i)) !== -1) {
 						++i;
+					}
 
 					// Create a sparse array
 					tokenStack[i] = match;

components/prism-mizar.js

@@ -9,4 +9,4 @@ Prism.languages.mizar = {
 	'number': /(?:\b|-)\d+\b/,
 	'operator': /\.\.\.|->|&|\.?=/,
 	'punctuation': /\(#|#\)|[,:;\[\](){}]/
-};
+};

components/prism-neon.js

@@ -38,4 +38,3 @@ Prism.languages.neon = {
 	},
 	'punctuation': /[,:=[\]{}()-]/,
 };
-

components/prism-php-extras.js

@@ -8,4 +8,4 @@ Prism.languages.insertBefore('php', 'variable', {
 			punctuation: /::|\\/
 		}
 	}
-});
+});

components/prism-soy.js

@@ -93,4 +93,4 @@
 		Prism.languages['markup-templating'].tokenizePlaceholders(env, 'soy');
 	});
 
-}(Prism));
+}(Prism));

components/prism-stylus.js

@@ -117,7 +117,6 @@
 		},
 
 
-
 		// A selector can contain parentheses only as part of a pseudo-element
 		// It can span multiple lines.
 		// It must end with a comma or an accolade or have indented content.

components/prism-v.js

@@ -76,4 +76,4 @@
 			}
 		}
 	});
-}(Prism));
+}(Prism));

components/prism-wasm.js

@@ -28,4 +28,4 @@ Prism.languages.wasm = {
 	'variable': /\$[\w!#$%&'*+\-./:<=>?@\\^_`|~]+/i,
 	'number': /[+-]?\b(?:\d(?:_?\d)*(?:\.\d(?:_?\d)*)?(?:[eE][+-]?\d(?:_?\d)*)?|0x[\da-fA-F](?:_?[\da-fA-F])*(?:\.[\da-fA-F](?:_?[\da-fA-D])*)?(?:[pP][+-]?\d(?:_?\d)*)?)\b|\binf\b|\bnan(?::0x[\da-fA-F](?:_?[\da-fA-D])*)?\b/,
 	'punctuation': /[()]/
-};
+};

components/prism-xojo.js

@@ -14,4 +14,4 @@ Prism.languages.xojo = {
 	'keyword': /\b(?:AddHandler|App|Array|As(?:signs)?|Auto|By(?:Ref|Val)|Boolean|Break|Byte|Call|Case|Catch|CFStringRef|CGFloat|Class|Color|Const|Continue|CString|Currency|CurrentMethodName|Declare|Delegate|Dim|Do(?:uble|wnTo)?|Each|Else(?:If)?|End|Enumeration|Event|Exception|Exit|Extends|False|Finally|For|Function|Get|GetTypeInfo|Global|GOTO|If|Implements|In|Inherits|Int(?:erface|eger|8|16|32|64)?|Lib|Loop|Me|Module|Next|Nil|Object|Optional|OSType|ParamArray|Private|Property|Protected|PString|Ptr|Raise(?:Event)?|ReDim|RemoveHandler|Return|Select(?:or)?|Self|Set|Single|Shared|Short|Soft|Static|Step|String|Sub|Super|Text|Then|To|True|Try|Ubound|UInt(?:eger|8|16|32|64)?|Until|Using|Var(?:iant)?|Wend|While|WindowPtr|WString)\b/i,
 	'operator': /<[=>]?|>=?|[+\-*\/\\^=]|\b(?:AddressOf|And|Ctype|IsA?|Mod|New|Not|Or|Xor|WeakAddressOf)\b/i,
 	'punctuation': /[.,;:()]/
-};
+};

dangerfile.js

@@ -45,7 +45,9 @@ const getChangedMinifiedFiles = async () => {
 
 // https://stackoverflow.com/questions/15900485/correct-way-to-convert-size-in-bytes-to-kb-mb-gb-in-javascript
 const formatBytes = (bytes, decimals = 2) => {
-	if (bytes === 0) return '0 Bytes';
+	if (bytes === 0) {
+		return '0 Bytes';
+	}
 
 	const k = 1000;
 	const dm = decimals < 0 ? 0 : decimals;

docs/index.html

@@ -63,6 +63,7 @@ <h2>Contribute to Prism!</h2>
 <ul>
 <li>Read the <a href="https://prismjs.com/extending.html">documentation</a>. Prism was designed to be extensible.</li>
 <li>Do not edit <code>prism.js</code>, it’s just the version of Prism used by the Prism website and is built automatically. Limit your changes to the unminified files in the <code>components/</code> folder. <code>prism.js</code> and all minified files are also generated automatically by our build system.</li>
+<li>Use <code>npm ci</code> to install Prism's dependencies. Do not use <code>npm install</code> because it will cause non-deterministic builds.</li>
 <li>The build system uses <a href="https://github.com/gulpjs/gulp">gulp</a> to minify the files and build <code>prism.js</code>. With all of Prism's dependencies installed, you just need to run the command <code>npm run build</code>.</li>
 <li>Please follow the code conventions used in the files already. For example, I use <a href="http://lea.verou.me/2012/01/why-tabs-are-clearly-superior/">tabs for indentation and spaces for alignment</a>. Opening braces are on the same line, closing braces on their own line regardless of construct. There is a space before the opening brace. etc etc.</li>
 <li>Please try to err towards more smaller PRs rather than a few huge PRs. If a PR includes changes that I want to merge and also changes that I don't, handling it becomes difficult.</li>

plugins/command-line/prism-command-line.js

@@ -29,6 +29,16 @@
 		return s;
 	}
 
+	/**
+	 * Returns whether the given hook environment has a command line info object.
+	 *
+	 * @param {any} env
+	 * @returns {boolean}
+	 */
+	function hasCommandLineInfo(env) {
+		var vars = env.vars = env.vars || {};
+		return 'command-line' in vars;
+	}
 	/**
 	 * Returns the command line info object from the given hook environment.
 	 *
@@ -129,6 +139,11 @@
 	});
 
 	Prism.hooks.add('complete', function (env) {
+		if (!hasCommandLineInfo(env)) {
+			// the previous hooks never ran
+			return;
+		}
+
 		var commandLine = getCommandLineInfo(env);
 
 		if (commandLine.complete) {