/FrameworkBenchmarks

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

Kelp framework fix #2427

Merged
merged 1 commit into from Dec 27, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@nbrady-techempower @LadyMozzarella
@nbrady-techempower
Member

nbrady-techempower commented Dec 27, 2016

Kelp's two tests use the same app.pl file which configures both databases. Because of this the mysql tests are failing without a mongo installation there. A quick fix here is to use one setup file that installs both db's. Would like to separate this out at some point.

@LadyMozzarella LadyMozzarella added the PR: LGTM if Travis-CI agrees! label Dec 27, 2016

@LadyMozzarella LadyMozzarella merged commit ac0ce08 into TechEmpower:master Dec 27, 2016
1 check failed

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details

@nbrady-techempower nbrady-techempower deleted the nbrady-techempower:perl-kelp branch Dec 28, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment