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

[ci] Upstream benchmark script changes from https://github.com/paritytech/substrate/pull/11493 #5586

Merged
merged 3 commits into from
May 30, 2022

Conversation

coderobe
Copy link
Contributor

@coderobe coderobe 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 May 23, 2022
@coderobe coderobe requested a review from a team as a code owner May 23, 2022 17:16
Copy link
Contributor

@chevdor chevdor left a comment

Choose a reason for hiding this comment

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

LGTM besides the missing eof

scripts/ci/run_benches_for_runtime.sh Outdated Show resolved Hide resolved
scripts/ci/run_benches_for_runtime.sh Outdated Show resolved Hide resolved
scripts/ci/run_benches_for_runtime.sh Outdated Show resolved Hide resolved
coderobe and others added 2 commits May 23, 2022 23:13
Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Co-authored-by: Chevdor <chevdor@users.noreply.github.com>
@paritytech-ci paritytech-ci requested a review from a team May 23, 2022 23:39
@coderobe
Copy link
Contributor Author

bot merge

@paritytech-processbot
Copy link

Error: Statuses failed for 84140d9

@coderobe
Copy link
Contributor Author

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 750fcd9 into master May 30, 2022
@paritytech-processbot paritytech-processbot bot deleted the coderobe/benchscript branch May 30, 2022 08:45
al3mart pushed a commit that referenced this pull request Jul 14, 2022
#5586)

* [ci] Upstream benchmark script changes from paritytech/substrate#11493

* Update scripts/ci/run_benches_for_runtime.sh

Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Update scripts/ci/run_benches_for_runtime.sh

Co-authored-by: Chevdor <chevdor@users.noreply.github.com>

Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Co-authored-by: Chevdor <chevdor@users.noreply.github.com>
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.

6 participants