Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Add additional pylint arguments field #255

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kapot65
Copy link

@kapot65 kapot65 commented Apr 20, 2018

No description provided.

@@ -1,7 +1,7 @@
{
"name": "linter-pylint",
"main": "./lib/main.js",
"version": "2.1.1",
Copy link
Member

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": {
Copy link
Member

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
}
}
}
}
Copy link
Member

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
Copy link
Member

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants