-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
Remove testing dependencies from application templates #2341
Comments
I am ok with this, but as for me i got my |
Well, another issue is that you need 512 MB of RAM to install it. |
not sure, could be so maybe, i have not measured this, on my pc i have 16GB, so such problems for me are just skiped. but as i said i am ok with this. |
|
Why keeping |
it can, you can download phar or install via composer. @qiangxue also note that we are using |
Makes sense. |
Perhaps we should suggest @davert to include them in phar? Right now they depend on PHPUnit package. If they are required via composer, it means PHPUnit package will be installed while codeception phar already includes PHPUnit. |
@qiangxue he is |
Ok, I agree pretty much testing dependencies for basic application. Especially when not too many developers will be interested in testing from start. Right now I work on codeception 2.0 which goes to master branch (as unstable development). I'm trying to do less BC breaks when possible, and last night I got everything back green on Travis, but still I should apologize for any issues that may happen, as you rely on the master branch. I promise to stabilize it before the Yii2 RC released.
Right now there is no phar package for Codeception 2.0. I'm going to build it when I have first release. Also you may put codeception dependencies into the |
I think that we should include Codeception into yii2-codeception extension. In my project yii2-codeception is in "require-dev" section of composer.json and we also can install project with "--no-dev" option. |
Section |
All fixed. |
Currently basic application has the following in dependencies:
It triggers pretty big download and isn't actually needed in lots of cases where the application is to be used.
I propose removing it from dependencies and instead putting it to readme.
Doing that we still need to keep tests running on travis.
The text was updated successfully, but these errors were encountered: