emacs-php / emacs-php/phpstan.el

[flycheck] Maybe remove :enabled check

Open
#2 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
Emacs Lisp
Stars
28
Forks
12
PR merge metrics
No merged PRs in 30d

Description

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?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.