Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix round initialization when in lazy loading #3005

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

RomarQ
Copy link
Contributor

@RomarQ RomarQ commented Oct 11, 2024

What does it do?

Fixes an error all members of CandidateQ must be candidates which happen after a completing a staking round when using lazy loading.

@RomarQ RomarQ self-assigned this Oct 11, 2024
@RomarQ RomarQ added the lazy-loading These changes affect the lazy loading feature label Oct 11, 2024
Copy link
Contributor

WASM runtime size check:

Compared to target branch

Moonbase runtime: 2260 KB (no changes) ✅

Moonbeam runtime: 2228 KB (no changes) ✅

Moonriver runtime: 2232 KB (no changes) ✅

Compared to latest release (runtime-3300)

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

Moonbeam runtime: 2228 KB (+232 KB compared to latest release) ⚠️

Moonriver runtime: 2232 KB (+240 KB compared to latest release) ⚠️

@RomarQ RomarQ added 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 labels Oct 11, 2024
Copy link
Contributor

Coverage Report

@@                        Coverage Diff                         @@
##           master   rq/fix-round-initialization-ll      +/-   ##
==================================================================
- Coverage   79.06%                           79.01%   -0.05%     
  Files         305                              305              
+ Lines       88382                            88446      +64     
==================================================================
  Hits        69878                            69878              
+ Misses      18504                            18568      +64     
Files Changed Coverage

Coverage generated Fri Oct 11 09:07:22 UTC 2024

@RomarQ RomarQ marked this pull request as ready for review October 11, 2024 09:27
@RomarQ RomarQ requested review from a team, gonzamontiel and Agusrodri and removed request for a team October 11, 2024 09:27
@RomarQ RomarQ merged commit f21e974 into master Oct 11, 2024
43 of 47 checks passed
@RomarQ RomarQ deleted the rq/fix-round-initialization-ll branch October 11, 2024 13:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
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) lazy-loading These changes affect the lazy loading feature not-breaking Does not need to be mentioned in breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants