Skip to content

Docblock updates.#18

Open
jasonmm wants to merge 1 commit intogajus:masterfrom
jasonmm:master
Open

Docblock updates.#18
jasonmm wants to merge 1 commit intogajus:masterfrom
jasonmm:master

Conversation

@jasonmm
Copy link
Copy Markdown

@jasonmm jasonmm commented Feb 26, 2016

This commit just updates some docblocks. I was getting some warnings in my IDE and this cleared those.
@jasonmm
Copy link
Copy Markdown
Author

jasonmm commented Feb 28, 2016

Travis CI is failing because one of the dependencies (satooshi/php-coveralls) only supports PHP 5.5 and above. There is a build environment for PHP 5.4 and that is the one that is failing. The PHP 5.5 and 5.6 environments are successful.
I'm not sure why the Coveralls is failing. The message even says the "decrease" in coverage was 0.0%. I've never used Coveralls before so maybe I am reading it wrong?
Can you let me know if there are any changes needed on my part to get this pull request accepted?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@jasonmm