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

Fix Rust support after recent changes #898

Merged
merged 5 commits into from
Aug 29, 2022
Merged

Fix Rust support after recent changes #898

merged 5 commits into from
Aug 29, 2022

Conversation

nazar-pc
Copy link
Collaborator

First commit restores code unnecessarily removed in #893 without which Rust can't send anything to worker.

Second commit fixes Rust support after #893, there I had to change how requests and notifications look like a bit, as the result we have less code and it is a bit cleaner.

Third commit updates dump data structure and tests according to changes in #894.

Last commit bumps versions, dependencies and updates changelog so we can publish it.

Copy link
Member

@ibc ibc left a comment

Choose a reason for hiding this comment

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

Thanks.

@ibc
Copy link
Member

ibc commented Aug 29, 2022

Any chance to have this merged tomorrow morning? I'd like to make a release for Node and I assume we need another one for Rust.

@nazar-pc
Copy link
Collaborator Author

It is just the comments that are left. I'll try to write some within next few hours.

@ibc
Copy link
Member

ibc commented Aug 29, 2022

I'll merge. Thanks for this.

@ibc ibc merged commit 6ac1cd2 into versatica:v3 Aug 29, 2022
@ibc
Copy link
Member

ibc commented Aug 29, 2022

3.6.10 released

@nazar-pc
Copy link
Collaborator Author

Rust 0.11.0 is out too

@nazar-pc nazar-pc deleted the fix-rust branch August 29, 2022 21:21
piranna pushed a commit to dyte-in/mediasoup that referenced this pull request Feb 9, 2023
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