add support for pylama linter

Author: DonJayamanne

package.json

@@ -804,6 +804,24 @@
             "type": "string"
           }
         },
+        "python.linting.pylamaEnabled": {
+          "type": "boolean",
+          "default": false,
+          "description": "Whether to lint Python files using pylama."
+        },
+        "python.linting.pylamaPath": {
+          "type": "string",
+          "default": "pylama",
+          "description": "Path to pylama, you can use a custom version of pylama by modifying this setting to include the full path."
+        },
+        "python.linting.pylamaArgs": {
+          "type": "array",
+          "description": "Arguments passed in. Each argument is a separate item in the array.",
+          "default": [],
+          "items": {
+            "type": "string"
+          }
+        },
         "python.unitTest.outputWindow": {
           "type": "string",
           "default": "Python Test Log",

src/client/linters/pylama.ts

@@ -4,6 +4,8 @@ import * as baseLinter from './baseLinter';
 import {OutputChannel} from 'vscode';
 import { Product } from '../common/installer';
 
+const REGEX = '(?<file>.py):(?<line>\\d+):(?<column>\\d+): \\[(?<type>\\w+)\\] (?<code>\\w\\d+) (?<message>.*)\\r?(\\n|$)';
+
 export class Linter extends baseLinter.BaseLinter {
     constructor(outputChannel: OutputChannel, workspaceRootPath: string) {
         super('pylama', Product.pylama, outputChannel, workspaceRootPath);
@@ -20,7 +22,7 @@ export class Linter extends baseLinter.BaseLinter {
         let pylamaPath = this.pythonSettings.linting.pylamaPath;
         let pylamaArgs = Array.isArray(this.pythonSettings.linting.pylamaArgs) ? this.pythonSettings.linting.pylamaArgs : [];
         return new Promise<baseLinter.ILintMessage[]>(resolve => {
-            this.run(pylamaPath, pylamaArgs.concat(['--format=%(row)d,%(col)d,%(code)s,%(code)s:%(text)s', filePath]), filePath, txtDocumentLines, this.workspaceRootPath).then(messages => {
+            this.run(pylamaPath, pylamaArgs.concat(['--format=parsable', filePath]), filePath, txtDocumentLines, this.workspaceRootPath, REGEX).then(messages => {
                 // All messages in pylama are treated as warnings for now
                 messages.forEach(msg => {
                     msg.severity = baseLinter.LintMessageSeverity.Information;