Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Linting does not use .puppet-lint.rc #43

Open
glennsarti opened this issue Jul 6, 2018 · 2 comments
Open

Linting does not use .puppet-lint.rc #43

glennsarti opened this issue Jul 6, 2018 · 2 comments
Labels
bug Something isn't working puppet-lint

Comments

@glennsarti
Copy link
Contributor

glennsarti commented Jul 6, 2018

Cross ref: puppetlabs/puppet-vscode#304

Original author: @setaou


What Versions are you running?

OS Version: Ubuntu 18.04
VSCode Version: 1.24.1
Puppet Extension Version: 0.10.0
Puppet Agent Version: 4.10.4

What You Are Seeing?

I have a .puppet-lint.rc file at the root of my workspace containing only the following line :
--no-140chars-check

If I use the puppet-lint command from a terminal, it uses this config file and ignores the warnings about long lines. However, the linting in vscode still shows the warnings.

What is Expected?

The linting in vscode should enforce the options set in .puppet-lint.rc.

@glennsarti
Copy link
Contributor Author

Added commentary


@glennsarti : Is this inside a module or control repository?
@setaou : It is at the root of the repository (which is the whole /etc/puppet).

@glennsarti
Copy link
Contributor Author

Hrmm ok...so it's neither a module or control repo.

So it looks like our change directory command isn't working anymore and we'll need to emulate - https://github.com/rodjek/puppet-lint/blob/41eec3b1d221ee0919af1b5e17adbf898fbb4432/lib/puppet-lint/optparser.rb#L139-L146

Probably a good idea to change this anyway as https://github.com/lingua-pupuli/puppet-editor-services/blob/master/lib/puppet-languageserver/manifest/validation_provider.rb#L41 isn't threadsafe

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working puppet-lint
Projects
None yet
Development

No branches or pull requests

2 participants