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

contracts: Fix failing benchmark test #7900

Merged
1 commit merged into from
Jan 14, 2021
Merged

contracts: Fix failing benchmark test #7900

1 commit merged into from
Jan 14, 2021

Conversation

athei
Copy link
Member

@athei athei commented Jan 14, 2021

The logic of this assert! was flawed. It slipped though the cracks because it was the result of two PRs that were merged at roughly the same time.

@athei athei 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 Jan 14, 2021
@athei athei requested a review from gui1117 January 14, 2021 16:34
@gui1117
Copy link
Contributor

gui1117 commented Jan 14, 2021

bot merge

@ghost
Copy link

ghost commented Jan 14, 2021

Waiting for commit status.

@ghost ghost merged commit d0b2e7d into master Jan 14, 2021
@ghost ghost deleted the at-fix-benchmark branch January 14, 2021 17:15
This pull request was closed.
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
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants