Add starting column number to violation output string (refs #52).

Author: DavidAnson

markdownlint.js

@@ -116,11 +116,12 @@ function prepareFileList(files, fileExtensions, previousResults) {
 }
 
 function printResult(lintResult) {
-  const results = flatten(Object.keys(lintResult).map(function (file) {
-    return lintResult[file].map(function (result) {
+  const results = flatten(Object.keys(lintResult).map(file => {
+    return lintResult[file].map(result => {
       return {
         file: file,
         lineNumber: result.lineNumber,
+        column: (result.errorRange && result.errorRange[0]) || 0,
         names: result.ruleNames.join('/'),
         description: result.ruleDescription +
           (result.errorDetail ? ' [' + result.errorDetail + ']' : '') +
@@ -130,12 +131,14 @@ function printResult(lintResult) {
   }));
   let lintResultString = '';
   if (results.length > 0) {
-    results.sort(function (a, b) {
+    results.sort((a, b) => {
       return a.file.localeCompare(b.file) || a.lineNumber - b.lineNumber ||
         a.names.localeCompare(b.names) || a.description.localeCompare(b.description);
     });
-    lintResultString = results.map(function (result) {
-      return result.file + ':' + result.lineNumber + ' ' + result.names + ' ' + result.description;
+    lintResultString = results.map(result => {
+      const {file, lineNumber, column, names, description} = result;
+      const columnText = column ? `:${column}` : '';
+      return `${file}:${lineNumber}${columnText} ${names} ${description}`;
     }).join('\n');
     // Note: process.exit(1) will end abruptly, interrupting asynchronous IO
     // streams (e.g., when the output is being piped). Just set the exit code

test/test.js

@@ -5,7 +5,7 @@ import path from 'path';
 import test from 'ava';
 import execa from 'execa';
 
-const errorPattern = /(\.md|\.markdown|\.mdf|stdin):\d+ MD\d{3}/gm;
+const errorPattern = /(\.md|\.markdown|\.mdf|stdin):\d+(:\d+)? MD\d{3}/gm;
 
 process.chdir('./test');
 
@@ -226,10 +226,10 @@ test('linting results are sorted by file/line/names/description', async t => {
       'incorrect.md:1 MD022/blanks-around-headings/blanks-around-headers Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "## header 2"]',
       'incorrect.md:1 MD041/first-line-heading/first-line-h1 First line in file should be a top level heading [Context: "## header 2"]',
       'incorrect.md:2 MD022/blanks-around-headings/blanks-around-headers Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Above] [Context: "# header"]',
-      'incorrect.md:5 MD014/commands-show-output Dollar signs used before commands without showing output [Context: "$ code"]',
-      'incorrect.md:11 MD014/commands-show-output Dollar signs used before commands without showing output [Context: "$ code"]',
-      'incorrect.md:17 MD014/commands-show-output Dollar signs used before commands without showing output [Context: "$ code"]',
-      'incorrect.md:23 MD014/commands-show-output Dollar signs used before commands without showing output [Context: "$ code"]',
+      'incorrect.md:5:1 MD014/commands-show-output Dollar signs used before commands without showing output [Context: "$ code"]',
+      'incorrect.md:11:1 MD014/commands-show-output Dollar signs used before commands without showing output [Context: "$ code"]',
+      'incorrect.md:17:1 MD014/commands-show-output Dollar signs used before commands without showing output [Context: "$ code"]',
+      'incorrect.md:23:1 MD014/commands-show-output Dollar signs used before commands without showing output [Context: "$ code"]',
       ''
     ].join('\n');
     t.deepEqual(error.stdout, '');
@@ -376,7 +376,7 @@ function getCwdConfigFileTest(extension) {
       t.fail();
     } catch (error) {
       const expected = [
-        'heading-dollar.md:1 MD026/no-trailing-punctuation Trailing punctuation in heading [Punctuation: \'$\']',
+        'heading-dollar.md:1:10 MD026/no-trailing-punctuation Trailing punctuation in heading [Punctuation: \'$\']',
         ''
       ].join('\n');
       t.deepEqual(error.stdout, '');
@@ -637,8 +637,8 @@ test('.markdownlintignore is applied correctly', async t => {
     t.fail();
   } catch (error) {
     const expected = [
-      'incorrect.md:1 MD047/single-trailing-newline Files should end with a single newline character',
-      'subdir/incorrect.markdown:1 MD047/single-trailing-newline Files should end with a single newline character',
+      'incorrect.md:1:8 MD047/single-trailing-newline Files should end with a single newline character',
+      'subdir/incorrect.markdown:1:8 MD047/single-trailing-newline Files should end with a single newline character',
       ''
     ].join('\n');
     t.deepEqual(error.stdout, '');
@@ -656,8 +656,8 @@ test('--ignore-path works with .markdownlintignore', async t => {
     t.fail();
   } catch (error) {
     const expected = [
-      'incorrect.md:1 MD047/single-trailing-newline Files should end with a single newline character',
-      'subdir/incorrect.markdown:1 MD047/single-trailing-newline Files should end with a single newline character',
+      'incorrect.md:1:8 MD047/single-trailing-newline Files should end with a single newline character',
+      'subdir/incorrect.markdown:1:8 MD047/single-trailing-newline Files should end with a single newline character',
       ''
     ].join('\n');
     t.deepEqual(error.stdout, '');
@@ -675,7 +675,7 @@ test('--ignore-path works with .ignorefile', async t => {
     t.fail();
   } catch (error) {
     const expected = [
-      'incorrect.markdown:1 MD047/single-trailing-newline Files should end with a single newline character',
+      'incorrect.markdown:1:8 MD047/single-trailing-newline Files should end with a single newline character',
       ''
     ].join('\n');
     t.deepEqual(error.stdout, '');