Support the "Info" category for flake8

[remcohaszing]

Environment data

• VS Code version: 1.22.2
• Extension version: 2018.3.1
• OS and version: Ubuntu 17.10
• Python version: 3.6
• Type of virtual environment used: N/A
• Relevant/affected Python packages and their versions: flake8==3.5.0 flake8-import-order==0.17.1

Actual behavior

Flake8 plugin configuration settings don’t match the settings schema in vscode, even though the settings do work.

Expected behavior

Flake8 plugin settings are not marked as unknown in the vscode settings file.

Steps to reproduce:

1. Install flake8

2. Install flake8-import-order

3. Open a file with the following contents:

   from foo import bar
   
   import sys

4. Notice how this triggers an info level problem

5. Add this line to your vscode settings:

     "python.linting.flake8CategorySeverity.I": "Warning",

6. Notice this gives a warning in the settings file

7. Save or reopen the Python file

8. Notice that the severity level has changed from info to warning

Logs

N/A

[brettcannon]

We should come up with a flexible mechanism for classifying flake8 issues/categories so that we can cover this need as well as any other future categories a plug-in might add.

Idea I head was a dict whose keys get put into a trie which we can the use to find the specified category or issue's classification.

[MetRonnie]

This ought to work if the flake8 plugin uses more than 1 letter for the prefix, e.g. flake8-simplify uses SIM, but

"python.linting.flake8CategorySeverity.SIM": "Warning",

does not work (you have to use .S instead)

[gschwaer]

Similar case for [flake8-]darglint. Flake8 is called with:

/usr/bin/python -m flake8 --format=%(row)d,%(col)d,%(code).1s,%(code)s:%(text)s ~/path/to/file.py

thus generates

44,1,D,DAR101:Missing parameter(s) in Docstring: - name

In case someone has the same issue: Put the following in the settings to remap these to errors:

"python.linting.flake8CategorySeverity.D": "Error", // remapping of darglint errors to proper vscode errors

But of course, if you have another flake-8 plugin that generates error codes starting with D, this will remap them as well.

[Avasam]

This ought to work if the flake8 plugin uses more than 1 letter for the prefix, e.g. flake8-simplify uses SIM, but

"python.linting.flake8CategorySeverity.SIM": "Warning",

does not work (you have to use .S instead)

To extend on this, I would like to be able to do

  "python.linting.flake8CategorySeverity.SIM": "Warning",
  "python.linting.flake8CategorySeverity.SIM9": "Information",

[karthiknadig]

Please try the flake8 extension, currently in pre-release. We are looking for feedback on it, please create issues on github for the extension: https://marketplace.visualstudio.com/items?itemName=ms-python.flake8 . This will be the eventual successor for the flake8 linting support in the extension.

One of the things you can do there is customize the severity, by individual error codes or categories.

[karthiknadig]

Closing this, since we have begun to process to migrate users over to flake8 extension. Plese try it out: https://marketplace.visualstudio.com/items?itemName=ms-python.flake8

[Avasam]

Relevant equivalent issue on the new repo. In case anyone comes across this. Go there instead: microsoft/vscode-flake8#7