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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Use upstream github.com/pion/ice dependency #55

Merged
merged 2 commits into from
Jan 24, 2022
Merged

fix: Use upstream github.com/pion/ice dependency #55

merged 2 commits into from
Jan 24, 2022

Conversation

kylecarbs
Copy link
Member

A forked version containing the following fix was used:
pion/ice#411

This has been merged upstream! 馃コ

A forked version containing the following fix was used:
pion/ice#411

This has been merged upstream! 馃コ
@kylecarbs kylecarbs self-assigned this Jan 24, 2022
@codecov
Copy link

codecov bot commented Jan 24, 2022

Codecov Report

Merging #55 (8b1fa4e) into main (a96cd3f) will decrease coverage by 0.16%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #55      +/-   ##
==========================================
- Coverage   71.39%   71.23%   -0.17%     
==========================================
  Files          60       60              
  Lines        2395     2395              
  Branches       30       30              
==========================================
- Hits         1710     1706       -4     
- Misses        534      537       +3     
- Partials      151      152       +1     
Flag Coverage 螖
unittest-go-macos-latest 66.27% <酶> (+0.10%) 猬嗭笍
unittest-go-ubuntu-latest 69.77% <酶> (-0.05%) 猬囷笍
unittest-go-windows-latest ?
unittest-js 74.91% <酶> (酶)
Impacted Files Coverage 螖
peer/conn.go 76.64% <0.00%> (-1.20%) 猬囷笍

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 a96cd3f...8b1fa4e. Read the comment docs.

@@ -84,7 +81,7 @@ require (
github.com/opencontainers/image-spec v1.0.2 // indirect
github.com/opencontainers/runc v1.0.2 // indirect
github.com/pion/dtls/v2 v2.1.0 // indirect
github.com/pion/ice/v2 v2.1.18 // indirect
github.com/pion/ice/v2 v2.1.19 // indirect
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

馃帀 Nice that we don't have to use the fork anymore. Thanks for contributing the fix @kylecarbs !

@kylecarbs kylecarbs enabled auto-merge (squash) January 24, 2022 16:54
@kylecarbs kylecarbs merged commit 52e50fc into main Jan 24, 2022
@kylecarbs kylecarbs deleted the icedep branch January 24, 2022 16:54
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.

None yet

2 participants