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

Fix inconsistent latest_block_hash #25

Merged
merged 2 commits into from
Sep 23, 2024
Merged

Conversation

boundless-forest
Copy link
Member

Cherry-pick polkadot-evm#1424

* Add `best_hash` method

* Update earliest tag matched value

* Fix CI tests

* Code clean

* Fix review comments

* Update comment

* Fix compile after solving conflicts

* Remove the broken changes

* Update trait bound

* Simplify trait bound
@hackfisher
Copy link

@boundless-forest There are conflicts at, which prevent ci format pass

Diff in /home/runner/work/frontier/frontier/frame/evm/src/tests.rs at line 241:

@boundless-forest
Copy link
Member Author

boundless-forest commented Sep 23, 2024

Diff in /home/runner/work/frontier/frontier/frame/evm/src/tests.rs at line 241:

I'm working on fixinging it (force push to keep the commit history clean). We should respect the CI in this repo in the future, the CI of the branch that we used is not maintained well.

@boundless-forest boundless-forest marked this pull request as ready for review September 23, 2024 09:31
@AurevoirXavier
Copy link
Member

Suggest using the same rust version as darwinia

@boundless-forest
Copy link
Member Author

boundless-forest commented Sep 23, 2024

Suggest using the same rust version as darwinia

nightly-2024-03-17 -> 1.77.0 (darwinia version), but nightly. stable toolchain fails to compile some piece of code.

@AurevoirXavier AurevoirXavier merged commit 1fb09fb into polkadot-v1.7.2 Sep 23, 2024
3 of 5 checks passed
@boundless-forest boundless-forest deleted the bear-pick-fix branch September 23, 2024 11:31
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.

3 participants