Skip to content

Conversation

@pawsengineer
Copy link
Member

I'm doing with sfu-ws and trying to improve the example with data-channel broadcasting.

Relates to #534

@coveralls
Copy link

coveralls commented Mar 21, 2019

Pull Request Test Coverage Report for Build 2617

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 81.386%

Totals Coverage Status
Change from base Build 2606: 0.0%
Covered Lines: 4088
Relevant Lines: 5023

💛 - Coveralls

@pawsengineer pawsengineer force-pushed the broadcast-datachannel branch from b3f9166 to ce0a800 Compare March 21, 2019 16:41
@Sean-Der
Copy link
Member

Hey @imalic3 really nice work!

You are hitting some errors in CI, you can see them here

To re-create those errors locally you can run the commands from our travis.yml

When travis-ci is green we can merge right away! Also will add you as a member of the pion organization and would love to continue improving things together :)

@pawsengineer pawsengineer force-pushed the broadcast-datachannel branch 3 times, most recently from 4e45fab to cae908a Compare March 22, 2019 02:46
@pawsengineer pawsengineer force-pushed the broadcast-datachannel branch from cae908a to 420b20f Compare March 22, 2019 02:51
@pawsengineer
Copy link
Member Author

@Sean-Der, the issue has been fixed.

Thank you again for your awesome work :).

Copy link
Member

@Sean-Der Sean-Der left a comment

Choose a reason for hiding this comment

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

Fantastic work, thanks for working on this!

@Sean-Der Sean-Der merged commit 768536c into pion:master Mar 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants