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

webrtc: add third-party sdp module #22557

Merged
merged 2 commits into from
Apr 2, 2020

Conversation

fippo
Copy link
Contributor

@fippo fippo commented Mar 31, 2020

Adds the third-party sdp module available from
https://www.npmjs.com/package/sdp
in version 2.12.0 and commit
958f22a94231e4c1ca651b4a7d53b0e32bb4a558
without tests or dependencies. This is going to be used by simulcast
tests and is generally useful in manipulating the SDP generated and
consumed by WebRTC.

See web-platform-tests/rfcs#46 for discussion
on how to add third-party javascript libraries.

Adds the third-party sdp module available from
  https://www.npmjs.com/package/sdp
in version 2.12.0 and commit
  958f22a94231e4c1ca651b4a7d53b0e32bb4a558
without tests or dependencies. This is going to be used by simulcast
tests and is generally useful in manipulating the SDP generated and
consumed by WebRTC.

See web-platform-tests/rfcs#46 for discussion
on how to add third-party javascript libraries.
@alvestrand
Copy link
Contributor

Can you take the text from the PR description and put it into a README.wpt file, so that people looking at this afterwards can figure out where it came from?
(I've seen README.chromium and similar files elsewhere, that's the reason for the suggested filename)

Copy link
Member

@jan-ivar jan-ivar left a comment

Choose a reason for hiding this comment

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

Lgtm once Harald's #22557 (comment) is addressed.

@fippo
Copy link
Contributor Author

fippo commented Mar 31, 2020

went for .md, didn't specify the version or commit hash since that is already part of the commit and having two would be duplicating things which would get out of sync.

Copy link
Contributor

@alvestrand alvestrand left a comment

Choose a reason for hiding this comment

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

README LGTM!
The important thing with the README is that it tells you where to look for updates from.

@jan-ivar jan-ivar merged commit 1c48b98 into web-platform-tests:master Apr 2, 2020
@fippo fippo deleted the webrtc-add-sdp branch April 2, 2020 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants