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

[flycheck] Maybe remove :enabled check #2

Open
Fuco1 opened this issue Apr 11, 2018 · 0 comments
Open

[flycheck] Maybe remove :enabled check #2

Fuco1 opened this issue Apr 11, 2018 · 0 comments

Comments

@Fuco1
Copy link
Member

Fuco1 commented Apr 11, 2018

Currently the logic looks for the phpstan.neon file to enable the checker.

I store this file inside app/ or tests/ directories with a bit different configurations for each scenario and thus the check would never trigger. I'm not sure if it's worth to have this check as phpstan also works without config and people often don't have any, especially on smaller projects.

What I do now is I pass the config via the -c option dynamically (I set this as buffer-local-variable in my php init hook). I have a pull request in the making that will add this option to flycheck as well.

WDYT?

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

No branches or pull requests

1 participant