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

Conversation

@ggwpez
Copy link
Member

@ggwpez ggwpez commented Mar 11, 2022

#1076 somehow broke the check-runtime-benchmarks CI after failing to merge as companion and being force merged.

MR#1076 somehow broke the check-runtime-benchmarks CI.

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez marked this pull request as ready for review March 11, 2022 20:06
@ggwpez ggwpez added A3-inprogress A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes and removed A3-inprogress labels Mar 11, 2022
@bkchr
Copy link
Member

bkchr commented Mar 11, 2022

It broke because you added the crate without enabling the feature.

Copy link
Member

@bkchr bkchr left a comment

Choose a reason for hiding this comment

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

We can merge this to make the CI work again. However, you should then prepare a new pr to address my late comments on your other pr.

@ggwpez
Copy link
Member Author

ggwpez commented Mar 11, 2022

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 4ee6287 into master Mar 11, 2022
@paritytech-processbot paritytech-processbot bot deleted the oty-ci-fix branch March 11, 2022 20:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants