Skip to content

Test with even larger values, move code to right place#11

Merged
spkrka merged 4 commits intomasterfrom
krka/tests
Feb 16, 2015
Merged

Test with even larger values, move code to right place#11
spkrka merged 4 commits intomasterfrom
krka/tests

Conversation

@spkrka
Copy link
Member

@spkrka spkrka commented Feb 15, 2015

No description provided.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.5%) to 83.53% when pulling 455b612 on krka/tests into dd05c19 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.62%) to 83.66% when pulling 455b612 on krka/tests into dd05c19 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+6.44%) to 89.47% when pulling 03ab85191c4352277b662d21e3d6166ed4801df5 on krka/tests into dd05c19 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+6.44%) to 89.47% when pulling 03ab85191c4352277b662d21e3d6166ed4801df5 on krka/tests into dd05c19 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.73%) to 84.76% when pulling c32c66b on krka/tests into dd05c19 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+2.77%) to 85.8% when pulling 316bdd7ace81000a1d38703d10c10d12e472ff3b on krka/tests into dd05c19 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+2.78%) to 85.81% when pulling 648ede68d02c5da17bc2af72539c2e8f9edc0636 on krka/tests into dd05c19 on master.

@spkrka spkrka force-pushed the krka/tests branch 2 times, most recently from 663c764 to e4c181c Compare February 15, 2015 19:50
@spkrka
Copy link
Member Author

spkrka commented Feb 15, 2015

@protocol7 Maybe I put too many changes in the same PR now... but it would be nice to have some review on the API change - it might leak too many abstractions i guess, but i couldn't think of a cleaner way to expose a more complete connection state

@coveralls
Copy link

Coverage Status

Coverage increased (+2.78%) to 85.81% when pulling 1361c21 on krka/tests into dd05c19 on master.

@protocol7
Copy link
Contributor

+1, merge away!

spkrka added a commit that referenced this pull request Feb 16, 2015
Test with even larger values, move code to right place
@spkrka spkrka merged commit 7622f4e into master Feb 16, 2015
@spkrka spkrka deleted the krka/tests branch February 16, 2015 07:11
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.

3 participants