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

Benchmark Slots and Auction #2181

Closed
wants to merge 9 commits into from
Closed

Conversation

shawntabrizi
Copy link
Member

in progress

@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Dec 30, 2020
@@ -1751,3 +1751,48 @@ mod tests {
});
}
}

#[cfg(feature = "runtime-benchmarks")]
Copy link
Contributor

Choose a reason for hiding this comment

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

this don't deserve a new file? this file is already > 1500 lines...

@coriolinus
Copy link
Contributor

Related to #1512 also?

@@ -530,8 +532,10 @@ decl_module! {
// Should have already begun. Remove the on-boarding entry and register the
// parachain for its immediate start.
<Onboarding<T>>::remove(&para_id);
let _ = T::Parachains::
// TODO: Probably best to make sure this doesn't fail or handle fail conditions.
Copy link
Member

Choose a reason for hiding this comment

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

stray TODO

@gavofyork gavofyork 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. and removed A0-please_review Pull request needs code review. B1-releasenotes labels Jan 18, 2021
@shawntabrizi
Copy link
Member Author

replaced elsewhere

@shawntabrizi shawntabrizi deleted the shawntabrizi-slots-weight branch June 15, 2021 20:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A3-in_progress Pull request is in progress. No review needed at this stage. C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants