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

dont print proof size#14729

Merged
paritytech-processbot[bot] merged 1 commit intoparitytech:masterfrom
xlc:benchmarking-cli
Aug 8, 2023
Merged

dont print proof size#14729
paritytech-processbot[bot] merged 1 commit intoparitytech:masterfrom
xlc:benchmarking-cli

Conversation

@xlc
Copy link
Contributor

@xlc xlc commented Aug 8, 2023

I don't know if this is a leftover debugging code or something but it is useless and broke our bencher bot

You can see here it is printing a lot of useless 2496 bytes in stdout

Please also backport this to 1.0.0 release branch

@liamaharon liamaharon requested a review from ggwpez August 8, 2023 05:28
@liamaharon liamaharon added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Aug 8, 2023
@ggwpez
Copy link
Member

ggwpez commented Aug 8, 2023

bot merge

@paritytech-processbot paritytech-processbot bot merged commit e1d529b into paritytech:master Aug 8, 2023
@xlc xlc deleted the benchmarking-cli branch August 9, 2023 01:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes 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.

3 participants