-
-
Notifications
You must be signed in to change notification settings - Fork 190
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
Contributing guide #103
Contributing guide #103
Conversation
|
||
## Travis | ||
|
||
We automatically run your pull request through [Travis CI](http://www.travis-ci.org) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest to use https://travis-ci.org
here, to avoid going through a useless redirection.
## Travis | ||
|
||
We automatically run your pull request through [Travis CI](http://www.travis-ci.org) | ||
against PHP 5.5, PHP 5.6, PHP 7 and HHVM. If you break the tests, we cannot merge your code, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would say against supported PHP versions
to avoid having to maintain this list when it changes
@@ -13,7 +13,8 @@ | |||
{"name": "Johannes Schmitt", "email": "schmittjoh@gmail.com"} | |||
], | |||
"require": { | |||
"php": "^5.5 || ^7.0" | |||
"php": "^5.5 || ^7.0", | |||
"doctrine/coding-standard": "~0.1@dev" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
must be a dev requirement. We don't want to install it in each project using the package
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot. Put it here by mistake - sure it's only dev requirement.
LGTM 👍 |
[resolves #102 ] Doctrine ORM CONTRIBUTING.md changed to collections context.