Skip to content

fix https://github.com/microsoft/vscode-eslint/issues/682#698

Merged
dbaeumer merged 2 commits intomicrosoft:masterfrom
salvofid:master
Aug 12, 2019
Merged

fix https://github.com/microsoft/vscode-eslint/issues/682#698
dbaeumer merged 2 commits intomicrosoft:masterfrom
salvofid:master

Conversation

@salvofid
Copy link
Contributor

eslint.lintTaskOptions added: this options allows to specify a string that is used as argument list during the task linting the whole workspace

this options allows to specify a string that is used as argument list during the task linting the whole workspace
@msftclas
Copy link

msftclas commented Jun 27, 2019

CLA assistant check
All CLA requirements met.

@dbaeumer
Copy link
Member

@salvofid thanks for the PR. See my comments above.

@webkonstantin
Copy link

LGTM?

@dbaeumer dbaeumer merged commit f03c4fd into microsoft:master Aug 12, 2019
@dbaeumer
Copy link
Member

I did some minor additional tweaks. Named the option eslint.lintTask.options and renamed eslint.provideLintTask to eslint.lintTask.enable.

If options are provided no . is added to the command line. It must be part of options.

dbaeumer added a commit that referenced this pull request Aug 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants