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

Emitting event when setting external validators #857

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

Conversation

chexware
Copy link
Contributor

@chexware chexware commented Feb 7, 2025

Setting external validators now emits an event

@chexware chexware added D3-trivial PR contains trivial changes in a runtime directory that do not require an audit B0-silent Changes should not be mentioned in any release notes not-breaking Does not need to be mentioned in breaking changes labels Feb 7, 2025
Copy link
Contributor

github-actions bot commented Feb 7, 2025

WASM runtime size check:

Compared to target branch

dancebox runtime: 1416 KB (no changes) ✅

flashbox runtime: 824 KB (no changes) ✅

dancelight runtime: 2168 KB (no changes) ✅

container chain template simple runtime: 1116 KB (no changes) ✅

container chain template frontier runtime: 1404 KB (no changes) ✅

Copy link
Contributor

github-actions bot commented Feb 7, 2025

Coverage Report

(master)

@@                                   Coverage Diff                                   @@
##           master   aleks-emit-event-on-external-validator-registration      +/-   ##
=======================================================================================
+ Coverage   65.97%                                                66.05%   +0.08%     
- Files         339                                                   335       -4     
- Lines       59084                                                 58856     -228     
=======================================================================================
- Hits        38978                                                 38872     -106     
- Misses      20106                                                 19984     -122     
Files Changed Coverage
/chains/orchestrator-relays/runtime/dancelight/src/genesis_config_presets.rs 60.03% (-0.06%)
/chains/orchestrator-relays/runtime/dancelight/src/lib.rs 68.61% (-0.45%)
/chains/orchestrator-relays/runtime/dancelight/src/tests/common/mod.rs 97.37% (-0.18%)
/chains/orchestrator-relays/runtime/dancelight/src/tests/slashes.rs 98.58% (-0.05%)
/pallets/external-validators-rewards/src/lib.rs 87.86% (-0.68%)

Coverage generated Mon Feb 10 17:19:01 UTC 2025

…pected event when more than max validators are passed as input

Signed-off-by: Aleksandar Brayanov <[email protected]>
…istration

# Conflicts:
#	typescript-api/src/dancelight/interfaces/augment-api-events.ts
#	typescript-api/src/dancelight/interfaces/lookup.ts
…dators through middleware contract for test_zombie_tanssi_relay_eth_bridge.ts

Signed-off-by: Aleksandar Brayanov <[email protected]>
@chexware chexware marked this pull request as ready for review February 12, 2025 15:12
…ssi_relay_eth_bridge.ts

Signed-off-by: Aleksandar Brayanov <[email protected]>
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 D3-trivial PR contains trivial changes in a runtime directory that do not require an audit 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