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

Benchmark Offences Pallet #5411

Merged
merged 20 commits into from
Apr 7, 2020
Merged

Benchmark Offences Pallet #5411

merged 20 commits into from
Apr 7, 2020

Conversation

marcio-diaz
Copy link
Contributor

@marcio-diaz marcio-diaz commented Mar 26, 2020

Add benchmark for report_offence. Part of #4876

@marcio-diaz marcio-diaz added the A3-in_progress Pull request is in progress. No review needed at this stage. label Mar 26, 2020
@marcio-diaz marcio-diaz added this to the 2.0 milestone Mar 26, 2020
@kianenigma
Copy link
Contributor

A good chunk of work will be added to in_initialize of the module in #4517. Should probably wait for it as well.

@marcio-diaz marcio-diaz added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes and removed A3-in_progress Pull request is in progress. No review needed at this stage. labels Mar 30, 2020
@marcio-diaz marcio-diaz mentioned this pull request Mar 30, 2020
14 tasks
frame/offences/Cargo.toml Outdated Show resolved Hide resolved
frame/staking/src/lib.rs Show resolved Hide resolved
frame/offences/src/lib.rs Outdated Show resolved Hide resolved
frame/offences/benchmarking/src/lib.rs Show resolved Hide resolved
@gavofyork gavofyork added A5-grumble and removed A0-please_review Pull request needs code review. labels Apr 3, 2020
@marcio-diaz marcio-diaz added A0-please_review Pull request needs code review. and removed A5-grumble labels Apr 6, 2020
@shawntabrizi shawntabrizi added A8-mergeoncegreen and removed A0-please_review Pull request needs code review. labels Apr 7, 2020
@gnunicorn gnunicorn merged commit 9f1bddf into master Apr 7, 2020
@gnunicorn gnunicorn deleted the md-benchmark-offences branch April 7, 2020 09:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants