Symplify: [EasyCodingStandard] Must provide source in check command even if paths are set in yaml

Created on 11 Jan 2020  路  4Comments  路  Source: symplify/symplify

The documentation states that I can set paths in my ecs.yaml file. For me, this created the expectation that I don't have to provide any sources for the ecs check CLI command. But when I do, it echos the --help page. It seems as though the sources parameter is still required.

Using ECS v7.2.0 on Windows 10 with PHP 7.4.1, running the command in PowerShell.

To reproduce:

  • Have some PHP files in the app/ and tests/ of your project directory.
  • List parameters in ecs.yam (also in project directory):
parameters:
    paths:
        - 'app'
        - 'tests'
  • cd to project dir.
  • Run vendor/bin/ecs check --config=ecs.yaml --set=psr12 in project directory.

Expected behavior: ecs performs check on all files in app/ and tests/.
Actual behavior: ecs shows --help info

All 4 comments

Hi, your expectations are correct.

Thing is, the README is always on master branch, regardless the latest tag.
7.2.0 is the last tag, released 6 days ago,

The "paths" option was added just 2 days ago, so it's only on @dev version now
https://github.com/Symplify/Symplify/compare/v7.2.0...master

I'll publish new release now (since it's really cool feature that people want), I just wanted to explain, what you can meet at any open-source project. README might be out of sync of last tag.

I didn't think of that, my apologies.
Thanks for the quick response!

It should be published in 10-15 minutes as 7.2.1

Was this page helpful?
0 / 5 - 0 ratings