Skip to content

Conversation

amihaiemil
Copy link
Owner

Implementation for #19

@coveralls
Copy link

Pull Request Test Coverage Report for Build 34

  • 29 of 29 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+9.02%) to 17.845%

Totals Coverage Status
Change from base Build 32: 9.02%
Covered Lines: 53
Relevant Lines: 297

💛 - Coveralls

1 similar comment
@coveralls
Copy link

Pull Request Test Coverage Report for Build 34

  • 29 of 29 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+9.02%) to 17.845%

Totals Coverage Status
Change from base Build 32: 9.02%
Covered Lines: 53
Relevant Lines: 297

💛 - Coveralls

@amihaiemil
Copy link
Owner Author

@rultor merge it

@rultor
Copy link
Collaborator

rultor commented Feb 4, 2018

@rultor merge it

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit 1707e97 into master Feb 4, 2018
@amihaiemil amihaiemil deleted the 19 branch February 4, 2018 19:28
@rultor
Copy link
Collaborator

rultor commented Feb 4, 2018

@rultor merge it

@amihaiemil Done! FYI, the full log is here (took me 2min)

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.

4 participants