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

companion for #8783 - jsonrpsee #4344

Merged
merged 30 commits into from
May 10, 2022
Merged

Conversation

niklasad1
Copy link
Member

@niklasad1 niklasad1 commented Nov 22, 2021

No description provided.

@niklasad1 niklasad1 added A3-in_progress Pull request is in progress. No review needed at this stage. B1-releasenotes 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. labels Nov 24, 2021
@niklasad1 niklasad1 marked this pull request as ready for review December 2, 2021 09:59
@github-actions github-actions bot added A0-please_review Pull request needs code review. and removed A3-in_progress Pull request is in progress. No review needed at this stage. labels Dec 2, 2021
@drahnr drahnr added this to the v0.9.19 milestone Feb 22, 2022
@niklasad1 niklasad1 force-pushed the dp-jsonrpsee-integration-companion branch from 5203666 to 24b0def Compare April 6, 2022 16:09
@coderobe coderobe modified the milestones: v0.9.19, v0.9.20 Apr 8, 2022
@niklasad1 niklasad1 added the A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. label Apr 19, 2022
@coderobe coderobe added this to the v0.9.21 milestone Apr 21, 2022
@@ -25,7 +25,7 @@ use std::collections::HashMap;

const DEFAULT_PROMETHEUS_PORT: u16 = 9616;

#[substrate_test_utils::test]
#[substrate_test_utils::test(flavor = "multi_thread")]
Copy link

Choose a reason for hiding this comment

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

@niklasad1 is it something like this that maybe leads to the issue with port 9944 being hardcoded in the test code being hit?

Copy link
Member Author

Choose a reason for hiding this comment

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

most likely, I have pushed the latest stuff that hopefully fixes that.

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot
Copy link

Merge cancelled due to error. Error: Statuses failed for 66113d3

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants