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

simplified unit testing #114

Merged
merged 2 commits into from
Nov 29, 2014
Merged

simplified unit testing #114

merged 2 commits into from
Nov 29, 2014

Conversation

romankonz
Copy link
Contributor

It was a bit complicated to run unit tests.
Also the travis-script is much simpler (hope it works, i'll fix that recently if not).
Now you can run "git clone .... && cd tests && composer install --dev && phpunit" without need for a skeleton application.

@romankonz
Copy link
Contributor Author

no, first try on travis does not work, i'll fix it now

@romankonz
Copy link
Contributor Author

Travis build is working now.
if you agree with this change i'll do some more related cleanup before merge (or i can do this in another PR)
After this change, it's much easier to contribute in my opinion because you can work on a standalone clone. But on the downside it has it's own vendor folder. That's why i'm waiting for feedback before i continue.

@Ocramius
Copy link
Member

More of this, please :-)

Ocramius added a commit that referenced this pull request Nov 29, 2014
@Ocramius Ocramius merged commit 55915b8 into doctrine:master Nov 29, 2014
@Ocramius Ocramius self-assigned this Nov 29, 2014
@Ocramius Ocramius added this to the 1.0.0 milestone Nov 29, 2014
@Ocramius
Copy link
Member

@romankonz and thanks! :D

@Ocramius Ocramius modified the milestones: 0.8.2, 1.0.0 Mar 31, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants