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

Updated composer.json to CakePHP 2.10.3 #2992

Merged
merged 1 commit into from
Oct 20, 2017
Merged

Updated composer.json to CakePHP 2.10.3 #2992

merged 1 commit into from
Oct 20, 2017

Conversation

therealahall
Copy link
Contributor

Tested running against tfb --mode verify --test cakephp

@NateBrady23
Copy link
Member

Hi @aaron-hall-7 Thanks for your first contribution! It looks like travis didn't like this. I'm rerunning it. You say this passed in your local environment? If you could post a log of that here, that would be great! We'll have to figure out why we can't get a pass on travis before we merge.

@therealahall
Copy link
Contributor Author

@nbrady-techempower I'm now seeing warnings (but no errors) when running the tests locally. I'll clean up these warnings on my end and push the changes again which may hopefully clean up the errors on Travis

@NateBrady23
Copy link
Member

@aaron-hall-7 Looks like it passed after the rerun. Feel free to open another PR if you have additional improvements and I'll merge this one now

@NateBrady23 NateBrady23 merged commit 1b8d557 into TechEmpower:master Oct 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants