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

Fix Scrutinizer.ci and add code coverage #158

Closed
wants to merge 4 commits into from
Closed

Fix Scrutinizer.ci and add code coverage #158

wants to merge 4 commits into from

Conversation

BernardoSilva
Copy link

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #157
License MIT
Doc PR --

Fixes #157

@daum
Copy link
Collaborator

daum commented Nov 8, 2016

@BernardoSilva are you still looking to have your updates merged in? If so, please update your PR to be compatible with the current version. From there can try to get it merged.

@BernardoSilva
Copy link
Author

Hi @daum , fixed the merge conflict.
Sorry for my long delay to reply :)

Hope it still add some value to get it merged.

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.

Scrutinizer status not displayed
2 participants