Skip to content

Conversation

@davidliu
Copy link
Contributor

@davidliu davidliu commented Feb 9, 2022

Note: PRs #11 and #12 were dropped as they were patches originally taken from the upstream, and are no longer needed.

Copy link
Member

@davidzhao davidzhao left a comment

Choose a reason for hiding this comment

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

Looks good! I've enabled individual commit merging for the repo. that might be a better way to move forward to keep them separate. wdyt?

@davidliu
Copy link
Contributor Author

What's individual commit merging, and how does that differ?

@davidzhao
Copy link
Member

it will keep all the commits in main individually, rather than all together. It might make it easier to cherry pick changes to future branches

@davidliu
Copy link
Contributor Author

Ah, instead of squashing, got it.

@davidliu davidliu merged commit 8c7139f into webrtc-sdk:m97_release Feb 10, 2022
@hiroshihorie
Copy link
Member

hiroshihorie commented Feb 10, 2022

Nice !! No more simulcast factory null checks for android 🙂

@davidliu
Copy link
Contributor Author

The null checks were originally from upstream so they're taken care of now.

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.

4 participants