-
Notifications
You must be signed in to change notification settings - Fork 443
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds support for 'caller_is_root' #2332
Conversation
@olahfemi There are some CI errors due to an unclosed delimiter. You can check this guide on how to run what the CI does locally. |
@cmichi, the unclosed delimiter has been fixed, and many of the local verification steps are passing now. The failed steps weren't as a result of my changes. What do you want us to do about that? Also, if my latest commit seems to have changed many unrelated files, it's because of the master merge and local verification. |
It's likely that you are using a different nightlyv version locally to execute the
And push the changes? |
Alternatively: can you allow me to edit your PR as described here? |
I've updated the nightly version, reran fmt and pushed the changes. Also, maintainers have edit access to the PR, so you can make edits if necessary. |
Summary
Closes #_
cargo-contract
orpallet-contracts
?This PR aims to add support for the
caller_is_root
function as requested by this issueDescription
Since the
caller_is_root
function is similar tocaller_is_origin
, I looked for places where the latter was implemented and did the same for the former.Checklist before requesting a review
CHANGELOG.md