Skip to content
This repository has been archived by the owner on Jul 1, 2023. It is now read-only.

Support only checking the files you changed with the -h option #76

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Support only checking the files you changed with the -h option #76

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
850c73d
Update Settings.php
paladox Sep 9, 2016
43a45c6
Update Settings.php
paladox Sep 9, 2016
c6f9ef4
part 2
paladox Sep 9, 2016
27dbaec
Update
paladox Sep 10, 2016
68af75d
Update
paladox Sep 10, 2016
68b1de8
Update Manager.php
paladox Sep 10, 2016
a39753a
Update Manager.php
paladox Sep 10, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/Settings.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ public static function parseArguments(array $arguments)
-m \
--first-parent \
--format=format: \
-- "${PATH_ARGS[@]}" | egrep -v '^$' || :";
-- "'*' . array('.php', '.phtml', '.php3', '.php4', '.php5')" | egrep -v '^$' || :";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do this line? It do concat string with array. I mean that a result of this expression have non-sence.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

egrep is deprecated use grep -E

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on second thought, you should probably refactor this to avoid use shell pipe formatting. what is you trying to filter out? can you give example? perhaps fixing --format or other arguments helps

otherwise filter it in php side. more readable

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PHP Parallel Lint is supported on Windows too and this does not work on this OS. So I think, this must be reimplemented in PHP.


foreach ($arguments as $argument) {
if ($argument{0} !== '-') {
Expand Down