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

Conversation

@kapot65
Copy link

@kapot65 kapot65 commented Apr 20, 2018

No description provided.

{
"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.

"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")?

}
}
}
}
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.

"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