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

contracts: Fix __unstable__ feature #8872

Merged
1 commit merged into from
May 20, 2021
Merged

contracts: Fix __unstable__ feature #8872

1 commit merged into from
May 20, 2021

Conversation

athei
Copy link
Member

@athei athei commented May 20, 2021

Followup for #8780 . Fixes one bug that went unnoticed.

  • The comparison was against the wrong module variable
  • The comparison was needlessly duplicated

@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. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. labels May 20, 2021
@athei athei requested review from ascjones and cmichi May 20, 2021 13:36
@athei
Copy link
Member Author

athei commented May 20, 2021

bot merge

@ghost
Copy link

ghost commented May 20, 2021

Trying merge.

@ghost ghost merged commit fc410e8 into master May 20, 2021
@ghost ghost deleted the at-fix-unstable branch May 20, 2021 20:42
nazar-pc pushed a commit to autonomys/substrate that referenced this pull request Aug 8, 2021
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. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit.
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants