Skip to content
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

evm proxy: allow identity precompile through NonTransfer proxy #2465

Merged
merged 1 commit into from
Aug 30, 2023

Conversation

librelois
Copy link
Collaborator

@librelois librelois commented Aug 30, 2023

What does it do?

Allow identity precompile to be called through EVM proxy NonTransfer

What important points reviewers should know?

Is there something left for follow-up PRs?

What alternative implementations were considered?

Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?

What value does it bring to the blockchain users?

@librelois librelois added B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit not-breaking Does not need to be mentioned in breaking changes labels Aug 30, 2023
@librelois librelois requested review from noandrea and nbaztec August 30, 2023 15:14
@librelois librelois mentioned this pull request Aug 30, 2023
28 tasks
@github-actions
Copy link
Contributor

Coverage generated "Wed Aug 30 15:52:34 UTC 2023":
https://d3ifz9vhxc2wtb.cloudfront.net/pulls/2465/html/index.html

Master coverage: 87.39%
Pull coverage:

@librelois librelois merged commit 2d0f1e0 into master Aug 30, 2023
@librelois librelois deleted the elois-idty-precompile-non-transfer branch August 30, 2023 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit not-breaking Does not need to be mentioned in breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants