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

Rename is_orbiter method to reference collator pool #2684

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

elfedy
Copy link
Contributor

@elfedy elfedy commented Feb 26, 2024

What does it do?

Renames is_orbiter to is_collator_pool_with_active_orbiter. Previous name may cause confusion because it is not actually checking if an address is an orbiter for the round, but instead if the address is an orbiter pool with an active orbiter on that round.

@elfedy elfedy added 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 labels Feb 26, 2024
Copy link
Contributor

Coverage Report

@@                     Coverage Diff                     @@
##           master   elfedy-rename-is-orbiter     +/-   ##
===========================================================
  Coverage   73.68%                     73.68%   0.00%     
  Files         232                        232             
+ Lines       72750                      72753      +3     
===========================================================
  Hits        53603                      53603             
+ Misses      19147                      19150      +3     
Files Changed Coverage
/pallets/moonbeam-orbiters/src/lib.rs 68.95% (-0.85%) 🔽

Coverage generated Mon Feb 26 16:55:47 UTC 2024

@elfedy elfedy merged commit 60cedb7 into master Feb 27, 2024
31 checks passed
@elfedy elfedy deleted the elfedy-rename-is-orbiter branch February 27, 2024 11:16
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants