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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove idl comment about rtcpTransport being at risk #1678

Merged
merged 1 commit into from Dec 7, 2017

Conversation

Projects
None yet
5 participants
@fippo
Copy link
Contributor

commented Dec 4, 2017

the linebreak is confusing and without checking the source its not
clear that rtcpTransport is at risk and not getCapabilities

atrisk

remove idl comment about rtcpTransport being at risk
the linebreak is confusing and without checking the source its not
clear that rtcpTransport is at risk and not getCapabilities

@fippo fippo requested a review from fluffy as a code owner Dec 4, 2017

@alvestrand

This comment has been minimized.

Copy link
Contributor

commented Dec 4, 2017

I think warning about feature-at-risk in the big red box further down the page is sufficient.
LGTM.

@aboba

This comment has been minimized.

Copy link
Contributor

commented Dec 5, 2017

LGTM. I put in the comment originally as a reminder to complete the feature-at-risk text, and forgot to remove it ;(

@aboba

aboba approved these changes Dec 5, 2017

@fluffy

fluffy approved these changes Dec 7, 2017

@stefhak

stefhak approved these changes Dec 7, 2017

@aboba aboba merged commit 13ef120 into w3c:master Dec 7, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
ipr PR deemed acceptable.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.