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

override default host-config when using the dev node #3056

Merged
merged 4 commits into from
Nov 25, 2024

Conversation

RomarQ
Copy link
Contributor

@RomarQ RomarQ commented Nov 24, 2024

What does it do?

Overrides the default host-configuration included in the relay proof when using the dev service.

@RomarQ RomarQ added B0-silent Changes should not be mentioned in any release notes D2-notlive PR doesn't change runtime code (so can't be audited) labels Nov 24, 2024
Copy link
Contributor

github-actions bot commented Nov 24, 2024

WASM runtime size check:

Compared to target branch

Moonbase runtime: 2260 KB (no changes) ✅

Moonbeam runtime: 2232 KB (no changes) ✅

Moonriver runtime: 2236 KB (no changes) ✅

Compared to latest release (runtime-3300)

Moonbase runtime: 2260 KB (+232 KB compared to latest release) ⚠️

Moonbeam runtime: 2232 KB (+236 KB compared to latest release) ⚠️

Moonriver runtime: 2236 KB (+244 KB compared to latest release) ⚠️

Copy link
Contributor

github-actions bot commented Nov 24, 2024

Coverage Report

@@                        Coverage Diff                        @@
##           master   rq/override-host-config-in-dev     +/-   ##
=================================================================
  Coverage   74.75%                           74.75%   0.00%     
  Files         369                              369             
+ Lines       94211                            94246     +35     
=================================================================
+ Hits        70424                            70453     +29     
+ Misses      23787                            23793      +6     
Files Changed Coverage
/node/service/src/lib.rs 62.02% (+0.60%) 🔼

Coverage generated Sun Nov 24 19:13:53 UTC 2024

@RomarQ RomarQ marked this pull request as ready for review November 24, 2024 11:42
node/service/src/lib.rs Outdated Show resolved Hide resolved
node/service/src/lib.rs Outdated Show resolved Hide resolved
@RomarQ RomarQ self-assigned this Nov 25, 2024
@RomarQ RomarQ added the B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes label Nov 25, 2024
@RomarQ RomarQ added the not-breaking Does not need to be mentioned in breaking changes label Nov 25, 2024
@RomarQ RomarQ merged commit 6e6bb1c into master Nov 25, 2024
42 of 43 checks passed
@RomarQ RomarQ deleted the rq/override-host-config-in-dev branch November 25, 2024 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B0-silent Changes should not be mentioned in any release notes B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes D2-notlive PR doesn't change runtime code (so can't be audited) not-breaking Does not need to be mentioned in breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants