Including and excluding parts of the same directory doesn't work about psalm HOT 7 CLOSED

So something like

<inspectFiles>
    <directory name="src" />
    <directory name="tests">
        <excludeFiles>
            <directory name="tests/Stubs" />
        </excludeFiles>
    </directory>
</inspectFiles>

from psalm.

Not a bug – you cannot exclude specific directories like that. But I could add support for that within the inspectFiles area. In the meantime you could put the files you do want to tests in a subdirectory of tests

from psalm.

It'd definitely be nice to have this, otherwise I've got to do:

<inspectFiles>
    <directory name="src" />
    <file name="tests/bootstrap.php" />
    <file name="tests/CompilerTest" />
    <file name="tests/ParserTest.php" />
    <directory name="tests/Parser" />
    <directory name="tests/Traits" />
</inspectFiles>

from psalm.

Yeah that might work. Why not support the example in the top of the ticket?

from psalm.

I like keeping the specific files that will be checked in a self-contained node.

Maybe includeFiles and excludeFiles could be deprecated in favour of onlyFiles and ignoreFiles, as the latter could then be used in both places. Also I think we could change inspectFiles to projectFiles while we're at it.

So the XML would look something like

<psalm>
  <projectFiles>
    <directory name="src" />
    <directory name="tests" />
    <ignoreFiles>
      <directory name="tests/Stubs" />
    </ignoreFiles>
  </projectFiles>

  <issueHandler>
    <InvalidReturnType>
      <onlyFiles>
        ...
      </onlyFiles>
    </InvalidReturnType>

    <MixedInferredReturnType>
      <ignoreFiles>
        ...
      </ignoreFiles>
    </MixedInferredReturnType>
  </issueHandler>
</psalm>

from psalm.

I like the idea of just moving it to projectFiles declaration.

from psalm.

The wiki not contain updates about that, but actually is not possible to use pattern for exclude files like:
<directory name="*lib*" /> <directory name="/vendor/*" />

from psalm.