Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[9.0.x] Analyse multiple path at same time did not work with console command #185

Closed
llaville opened this issue Feb 21, 2023 · 2 comments
Closed
Assignees
Labels

Comments

@llaville
Copy link
Collaborator

New Issue

Diagnose

All distributions (source code via Composer installation, Docker image, PHAR version) are impacted by this issue.

Summary

Analyse multiple path at same time did not work with console command

Expected behaviour

Be able to scan multiple directories at same time

Actual behaviour

Only first directory is scanned

bin/phplint --no-configuration --ignore-exit-code -vvv src/ examples/ tests/
phplint 9.0.3 by overtrue and contributors.

Runtime       : PHP 8.1.14
Configuration : No config file loaded

┌──────────────────┬──────────────────┬─────────┐
│ Name             │ Value            │ Help    │
├──────────────────┼──────────────────┼─────────┤
│ path             │ ["src/"]         │ command │
├──────────────────┼──────────────────┼─────────┤
│ configuration    │ ".phplint.yml"   │ command │
│ no-configuration │ true             │ command │
│ exclude          │ []               │ command │
│ extensions       │ ["php"]          │ command │
│ jobs             │ 5                │ command │
│ no-cache         │ false            │ command │
│ cache            │ ".phplint.cache" │ command │
│ no-progress      │ false            │ command │
│ progress         │ "printer"        │ command │
│ log-json         │ null             │ command │
│ log-junit        │ null             │ command │
│ warning          │ false            │ command │
│ memory-limit     │ "512M"           │ command │
│ ignore-exit-code │ true             │ command │
└──────────────────┴──────────────────┴─────────┘


Time: < 1 sec, Memory: 6.0 MiB, Cache: 34 hits, 0 miss


 [OK] 34 files

@llaville
Copy link
Collaborator Author

Works as expected when using multiple paths with configuration YAML file :

path:
  - ./src
  - ./tests
jobs: 10
extensions:
  - php
exclude:
  - vendor
warning: true
memory-limit: -1
no-cache: false
log-json: false
log-junit: false

Running command (from PHPLint Git clone copy of source code) :

bin/phplint -vvv

Gave following results :

multiple-paths
multiple-paths-results

llaville added a commit that referenced this issue Feb 22, 2023
@llaville
Copy link
Collaborator Author

Fix results by image 😄

fix-issue-185

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant