Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

[Substrate Companion] Add WebRTC transport #6529

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

[Substrate Companion] Add WebRTC transport #6529

wants to merge 12 commits into from

Conversation

melekes
Copy link
Contributor

@melekes melekes commented Jan 10, 2023 •
edited
Loading

substrate companion: paritytech/substrate#12529

@melekes melekes mentioned this pull request Jan 10, 2023
Open
@melekes melekes added B1-releasenotes C1-low PR touches the given topic and has a low impact on builders. B0-silent Changes should not be mentioned in any release notes and removed B1-releasenotes B0-silent Changes should not be mentioned in any release notes labels Jan 10, 2023
@melekes melekes requested review from altonen and a team January 10, 2023 07:42
@melekes melekes self-assigned this Jan 10, 2023
@melekes
Copy link
Contributor Author

melekes commented Jan 10, 2023

tests are passing for me locally, so not sure what I'm doing wrong...

@melekes melekes changed the title [client/network] Add WebRTC transport [Substrate Companion] Add WebRTC transport Jan 10, 2023
@dmitry-markin
Copy link
Contributor

dmitry-markin commented Jan 10, 2023 •
edited
Loading

tests are passing for me locally, so not sure what I'm doing wrong...

Unfortunately, it's just the way companion system works: polkadot builds know nothing about substrate companions and try to build against substrate master. You should rely on continuous-integration/gitlab-check-dependent-polkadot test in substrate PR instead of the tests here.

@melekes
Copy link
Contributor Author

melekes commented Jan 17, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@the-right-joyce the-right-joyce added A0-please_review Pull request needs code review. B1-note_worthy Changes should be noted in the release notes and removed A0-pleasereview labels Jan 20, 2023
@melekes
Copy link
Contributor Author

melekes commented Jan 26, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@melekes
Copy link
Contributor Author

melekes commented Feb 12, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@melekes
Copy link
Contributor Author

melekes commented Mar 17, 2023

bot rebase

@paritytech-processbot
Copy link

Branch is already up-to-date

@melekes
Copy link
Contributor Author

melekes commented May 30, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@melekes
Copy link
Contributor Author

melekes commented May 31, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@melekes
Copy link
Contributor Author

melekes commented Jun 5, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@melekes melekes added the D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. label Jun 5, 2023
@melekes melekes requested a review from a team June 5, 2023 09:33
@melekes melekes added the T0-node This PR/Issue is related to the topic “node”. label Jun 5, 2023
@melekes
Copy link
Contributor Author

melekes commented Jun 7, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@melekes
Copy link
Contributor Author

melekes commented Jul 25, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@melekes
Copy link
Contributor Author

melekes commented Jul 27, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/3271091

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@altonen altonen altonen approved these changes

@dmitry-markin dmitry-markin dmitry-markin approved these changes

@kpp kpp kpp approved these changes

At least 1 approving review is required to merge this pull request.

Assignees

@melekes melekes

Labels
A0-please_review Pull request needs code review. B1-note_worthy Changes should be noted in the release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. T0-node This PR/Issue is related to the topic “node”.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@melekes @dmitry-markin @paritytech-cicd-pr @kpp @altonen @the-right-joyce