Update to version 0.20.2.

README.md

@@ -861,6 +861,7 @@ See [CONTRIBUTING.md](CONTRIBUTING.md) for more information.
            improve MD005/MD007/MD013/MD018/MD029/MD031/MD034/MD037/MD038/MD039, improve HTML
            comment handling, update dependencies.
   * 0.20.1 - Fix regression in MD037.
+  * 0.20.2 - Fix regression in MD037, improve MD038.
 
 [npm-image]: https://img.shields.io/npm/v/markdownlint.svg
 [npm-url]: https://www.npmjs.com/package/markdownlint

demo/markdownlint-browser.js

@@ -36,6 +36,8 @@ module.exports.inlineCommentRe = inlineCommentRe;
 module.exports.bareUrlRe = /(?:http|ftp)s?:\/\/[^\s\]"']*/ig;
 module.exports.listItemMarkerRe = /^([\s>]*)(?:[*+-]|\d+[.)])\s+/;
 module.exports.orderedListItemMarkerRe = /^[\s>]*0*(\d+)[.)]/;
+// Regular expression for emphasis markers
+var emphasisMarkersRe = /[_*]+/g;
 // readFile options for reading with the UTF-8 encoding
 module.exports.utf8Encoding = { "encoding": "utf8" };
 // All punctuation characters (normal and full-width)
@@ -328,8 +330,13 @@ module.exports.forEachHeading = function forEachHeading(params, handler) {
         }
     });
 };
-// Calls the provided function for each inline code span's content
+/**
-module.exports.forEachInlineCodeSpan =
+ * Calls the provided function for each inline code span's content.
+ *
+ * @param {string} input Markdown content.
+ * @param {Function} handler Callback function.
+ * @returns {void}
+ */
 function forEachInlineCodeSpan(input, handler) {
     var currentLine = 0;
     var currentColumn = 0;
@@ -409,7 +416,8 @@ module.exports.forEachInlineCodeSpan =
             currentColumn = startColumn;
         }
     }
-    };
+}
+module.exports.forEachInlineCodeSpan = forEachInlineCodeSpan;
 /**
  * Adds a generic error object via the onError callback.
  *
@@ -473,6 +481,37 @@ module.exports.frontMatterHasTitle =
         return !ignoreFrontMatter &&
             frontMatterLines.some(function (line) { return frontMatterTitleRe.test(line); });
     };
+/**
+ * Returns a list of emphasis markers in code spans.
+ *
+ * @param {Object} params RuleParams instance.
+ * @returns {number[][]} List of markers.
+ */
+function emphasisMarkersInCodeSpans(params) {
+    var lines = params.lines;
+    var byLine = new Array(lines.length);
+    filterTokens(params, "inline", function (token) {
+        var children = token.children, lineNumber = token.lineNumber, map = token.map;
+        if (children.some(function (child) { return child.type === "code_inline"; })) {
+            var tokenLines = lines.slice(map[0], map[1]);
+            forEachInlineCodeSpan(tokenLines.join("\n"), function (code, lineIndex, column, tickCount) {
+                var codeLines = code.split(newLineRe);
+                codeLines.forEach(function (codeLine, codeLineIndex) {
+                    var match = null;
+                    while ((match = emphasisMarkersRe.exec(codeLine))) {
+                        var byLineIndex = lineNumber - 1 + lineIndex + codeLineIndex;
+                        var inLine = byLine[byLineIndex] || [];
+                        var codeLineOffset = codeLineIndex ? 0 : column - 1 + tickCount;
+                        inLine.push(codeLineOffset + match.index);
+                        byLine[byLineIndex] = inLine;
+                    }
+                });
+            });
+        }
+    });
+    return byLine;
+}
+module.exports.emphasisMarkersInCodeSpans = emphasisMarkersInCodeSpans;
 /**
  * Gets the most common line ending, falling back to the platform default.
  *
@@ -2702,7 +2741,7 @@ module.exports = {
 },{"../helpers":2}],37:[function(require,module,exports){
 // @ts-check
 "use strict";
-var _a = require("../helpers"), addErrorContext = _a.addErrorContext, forEachLine = _a.forEachLine, isBlankLine = _a.isBlankLine;
+var _a = require("../helpers"), addErrorContext = _a.addErrorContext, emphasisMarkersInCodeSpans = _a.emphasisMarkersInCodeSpans, forEachLine = _a.forEachLine, includesSorted = _a.includesSorted, isBlankLine = _a.isBlankLine;
 var lineMetadata = require("./cache").lineMetadata;
 var emphasisRe = /(^|[^\\])(?:(\*\*?\*?)|(__?_?))/g;
 var asteriskListItemMarkerRe = /^(\s*)\*(\s+)/;
@@ -2714,11 +2753,12 @@ module.exports = {
     "tags": ["whitespace", "emphasis"],
     "function": function MD037(params, onError) {
         // eslint-disable-next-line init-declarations
-        var effectiveEmphasisLength, emphasisIndex, emphasisLength, pendingError;
+        var effectiveEmphasisLength, emphasisIndex, emphasisKind, emphasisLength, pendingError = null;
         // eslint-disable-next-line jsdoc/require-jsdoc
         function resetRunTracking() {
             emphasisIndex = -1;
             emphasisLength = 0;
+            emphasisKind = "";
             effectiveEmphasisLength = 0;
             pendingError = null;
         }
@@ -2761,6 +2801,7 @@ module.exports = {
             return null;
         }
         // Initialize
+        var ignoreMarkersByLine = emphasisMarkersInCodeSpans(params);
         resetRunTracking();
         forEachLine(lineMetadata(), function (line, lineIndex, inCode, onFence, inTable, inItem, onBreak) {
             var onItemStart = (inItem === 1);
@@ -2779,15 +2820,23 @@ module.exports = {
             var match = null;
             // Match all emphasis-looking runs in the line...
             while ((match = emphasisRe.exec(line))) {
+                var ignoreMarkersForLine = ignoreMarkersByLine[lineIndex] || [];
                 var matchIndex = match.index + match[1].length;
+                if (includesSorted(ignoreMarkersForLine, matchIndex)) {
+                    // Ignore emphasis markers inside code spans
+                    continue;
+                }
                 var matchLength = match[0].length - match[1].length;
+                var matchKind = (match[2] || match[3])[0];
                 if (emphasisIndex === -1) {
                     // New run
                     emphasisIndex = matchIndex + matchLength;
                     emphasisLength = matchLength;
+                    emphasisKind = matchKind;
                     effectiveEmphasisLength = matchLength;
                 }
-                else if (matchLength === effectiveEmphasisLength) {
+                else if ((matchLength === effectiveEmphasisLength) &&
+                    (matchKind === emphasisKind)) {
                     // Ending an existing run, report any pending error
                     if (pendingError) {
                         addErrorContext.apply(void 0, pendingError);
@@ -2830,7 +2879,7 @@ module.exports = {
 var _a = require("../helpers"), addErrorContext = _a.addErrorContext, filterTokens = _a.filterTokens, forEachInlineCodeSpan = _a.forEachInlineCodeSpan, newLineRe = _a.newLineRe;
 var leftSpaceRe = /^\s([^`]|$)/;
 var rightSpaceRe = /[^`]\s$/;
-var singleLeftRightSpaceRe = /^\s\S+\s$/;
+var singleLeftRightSpaceRe = /^\s(?:\S.*\S|\S)\s$/;
 module.exports = {
     "names": ["MD038", "no-space-in-code"],
     "description": "Spaces inside code span elements",

helpers/package.json

@@ -1,6 +1,6 @@
 {
   "name": "markdownlint-rule-helpers",
-  "version": "0.8.0",
+  "version": "0.9.0",
   "description": "A collection of markdownlint helper functions for custom rules",
   "main": "helpers.js",
   "author": "David Anson (https://dlaa.me/)",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "markdownlint",
-  "version": "0.20.1",
+  "version": "0.20.2",
   "description": "A Node.js style checker and lint tool for Markdown/CommonMark files.",
   "main": "lib/markdownlint.js",
   "types": "lib/markdownlint.d.ts",