Skip to content
This repository was archived by the owner on May 11, 2025. It is now read-only.

Partial rollback#172

Open
syerrapragada wants to merge 5 commits into
otalk:masterfrom
syerrapragada:partial_rollback
Open

Partial rollback#172
syerrapragada wants to merge 5 commits into
otalk:masterfrom
syerrapragada:partial_rollback

Conversation

@syerrapragada

Copy link
Copy Markdown
Contributor

@fippo Draft version of Rollback implementation. I want your feedback on this PR.

@codecov-io

codecov-io commented Nov 2, 2018

Copy link
Copy Markdown

Codecov Report

Merging #172 into master will decrease coverage by 0.08%.
The diff coverage is 99.09%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #172      +/-   ##
==========================================
- Coverage   99.81%   99.72%   -0.09%     
==========================================
  Files           8        8              
  Lines        1072     1104      +32     
==========================================
+ Hits         1070     1101      +31     
- Misses          2        3       +1
Impacted Files Coverage Δ
util.js 100% <ø> (ø) ⬆️
writemediasection.js 100% <100%> (ø) ⬆️
rtcpeerconnection.js 99.62% <98.97%> (-0.12%) ⬇️

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 529b503...d1491e6. Read the comment docs.

@fippo

fippo commented Nov 5, 2018

Copy link
Copy Markdown
Member

heads-up, I am probably still deep into making a new adapter release this week so might take some time until I can review :-|

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants