You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I would like to recommend a possible feature in terms of package:test --parallel command. I noticed that in your latest updates, you add an extra functionality in terms of caching which requires loading the proper phpunit.xml file (6fc0c47).
However, the code enforces the file's path, considering that by default it would be in the laravel's base path.
Although, there are external packages which extend your class and their configurations files (phpunit.xml) have different paths or maybe do not exist.
So, my suggestion is, to let others to overwrite the required file (if they want).
I was thinking a solution like this :
The above, will help external packages, which extend the nunomaduro/collision, to keep execute package:test --parallel command on their own configuration options.
Thank you
The text was updated successfully, but these errors were encountered:
Hello,
I would like to recommend a possible feature in terms of
package:test --parallel
command. I noticed that in your latest updates, you add an extra functionality in terms of caching which requires loading the properphpunit.xml
file (6fc0c47).However, the code enforces the file's path, considering that by default it would be in the laravel's base path.
Although, there are external packages which extend your class and their configurations files (
phpunit.xml
) have different paths or maybe do not exist.So, my suggestion is, to let others to overwrite the required file (if they want).
I was thinking a solution like this :
#273
The above, will help external packages, which extend the
nunomaduro/collision
, to keep executepackage:test --parallel
command on their own configuration options.Thank you
The text was updated successfully, but these errors were encountered: