Skip to content

chore(plugin/evm): remove duplicate libevm/core/types import in test file #926

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

riskrose
Copy link

Why this should be merged

remove duplicate package imports

How this works

How this was tested

Need to be documented?

Need to update RELEASES.md?

@riskrose riskrose requested review from ceyonur and a team as code owners April 18, 2025 07:59
@qdm12 qdm12 changed the title chore: remove duplicate package imports chore(plugin/evm): remove duplicate libevm/core/types import Apr 18, 2025
@qdm12 qdm12 changed the title chore(plugin/evm): remove duplicate libevm/core/types import chore(plugin/evm): remove duplicate libevm/core/types import in test file Apr 18, 2025
@ceyonur ceyonur enabled auto-merge April 18, 2025 16:03
@riskrose
Copy link
Author

Thanks for your review!

@qdm12
Copy link
Contributor

qdm12 commented Apr 23, 2025

Can you add a signature to your commit?

You can configure it following https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

and then use

git commit -S --amend --no-edit
git push --force

to override the previous commit with a new signed commit. Thanks! 👍

auto-merge was automatically disabled April 29, 2025 07:31

Head branch was pushed to by a user without write access

@riskrose
Copy link
Author

Can you add a signature to your commit?

You can configure it following https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

and then use

git commit -S --amend --no-edit
git push --force

to override the previous commit with a new signed commit. Thanks! 👍

Thanks

Rebased and Signed.

Please review again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants