Skip to content

Conversation

@dbrgn
Copy link
Member

@dbrgn dbrgn commented Nov 13, 2017

The old one confused me a bit when implementing the new client.

@dbrgn dbrgn requested a review from lgrahl November 13, 2017 13:14
@lgrahl lgrahl merged commit 4b2a340 into master Nov 13, 2017
@lgrahl lgrahl deleted the client-hello branch November 13, 2017 13:19
@codecov-io
Copy link

codecov-io commented Nov 13, 2017

Codecov Report

Merging #66 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #66   +/-   ##
=======================================
  Coverage   76.76%   76.76%           
=======================================
  Files           9        9           
  Lines        1278     1278           
  Branches      153      153           
=======================================
  Hits          981      981           
  Misses        250      250           
  Partials       47       47
Impacted Files Coverage Δ
saltyrtc/server/server.py 84.35% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9b2a2ee...48bfa2b. Read the comment docs.

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.

5 participants