-
Notifications
You must be signed in to change notification settings - Fork 48
Add additional pylint arguments field #255
base: master
Are you sure you want to change the base?
Conversation
@@ -1,7 +1,7 @@ | |||
{ | |||
"name": "linter-pylint", | |||
"main": "./lib/main.js", | |||
"version": "2.1.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please don't change the version in a PR, that will be done when releasing an update.
@@ -20,36 +20,42 @@ | |||
"description": "Command or full path to pylint. Use %p for current project directory (no trailing /) or %h for current project name.", | |||
"order": 1 | |||
}, | |||
"additionalArgs": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is automatically translated into a title, currently this would be "Additional Args". Can you change this to additionalArguments
("Additional Arguments")?
@@ -93,4 +99,4 @@ | |||
"node": true | |||
} | |||
} | |||
} | |||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please leave the trailing newline.
"pythonPath": { | ||
"type": "string", | ||
"default": "", | ||
"description": "Paths to be added to $PYTHONPATH. Use %p for current project directory or %f for the directory of the current file.", | ||
"order": 1 | ||
"order": 3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The order
was being used as more of a priority, please leave the rest of these settings as they were. 2
is fine for the value of your new setting.
No description provided.