Deep Static Analysis and Unit Testing about airship HOT 11 CLOSED

Fixed in vimeo/psalm@f096468

from airship.

You can do psalm --config=somefile.xml

from airship.

Relatedly, this curious chunk of code was flagged by latest Psalm dev-master.

from airship.

Oh, I should probably change null to []? (Composer still hasn't updated.)

from airship.

more the if (!\is_array($data)) { check on something that is definitely an array (because of the cast)

from airship.

Is there any chance we can just make the is_*() functions excluded from type errors?

from airship.

They should be excluded when the type comes from a docblock (as opposed to language-level check). I think the difference here is that the if is inside a loop, and I imagine I introduced a regression with vimeo/psalm#115. Will fix momentarily.

But still, how can that if statement ever be true?

from airship.

It can't ever be true; it's dead code. I'm removing it in a few seconds. :)

from airship.

Left to do before this can be closed (mostly meant as a note to myself so the next time I look through open issues for things to work on, I don't need to try to refresh my own memory):

• Find out if Psalm can be coerced into specifying a different .xml file from its CLI arguments.
  • If yes, create a psalm.xml for each Cabin.
  • If no, rename the existing psalm.xml file and rapidly create/remove symlinks to make it behave. Also, create a psalm.sh script to facilitate this for non-Travis testing.

from airship.

Perfect! Thanks. :)

from airship.

If you're running --diff builds locally you'll need to specify different cache directories for them, but I think your codebase is small enough that --diff builds aren't necessary.

from airship.