You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm a bit confused about the usage of the phpstan-working-dir variable.
I'm trying to setup phpstan.el so that I don't have to have a phpstan.neon file in a project.
I got this working by setting (setq phpstan-working-dir ".") so that the enabled check passes, however this seems very hacky.
Could you explain to me the use of phpstan-working-dir? The documentation mentions it should be different than the project root but doesn't give an explicit example.
In addition, I think I've found a bug while trying to figure out the use case of phpstan-working-dir. I tried the same "hack" (not having a phpstan.neon file and setting (setq phpstan-working-dir ".")) when using Docker.
In this case phpstan-get-config-file returns NIL which is passed to phpstan-normalize-path however since prefix is set to /app the function throws an error
Hi, thanks for this great package!
I'm a bit confused about the usage of the
phpstan-working-dir
variable.I'm trying to setup phpstan.el so that I don't have to have a
phpstan.neon
file in a project.I got this working by setting
(setq phpstan-working-dir ".")
so that the enabled check passes, however this seems very hacky.Could you explain to me the use of
phpstan-working-dir
? The documentation mentions it should be different than the project root but doesn't give an explicit example.In addition, I think I've found a bug while trying to figure out the use case of
phpstan-working-dir
. I tried the same "hack" (not having aphpstan.neon
file and setting(setq phpstan-working-dir ".")
) when using Docker.In this case
phpstan-get-config-file
returnsNIL
which is passed tophpstan-normalize-path
however sinceprefix
is set to/app
the function throws an errorThis is easily fixed by changing the function:
I'm happy to send a PR to fix the above issue however, it seems quite hacky.
The text was updated successfully, but these errors were encountered: