Skip to content

Conversation

@junichi-tanaka
Copy link
Contributor

@junichi-tanaka junichi-tanaka commented Feb 17, 2025

@junichi-tanaka junichi-tanaka self-assigned this Feb 17, 2025
.PHONY: fmt
fmt:
@cargo fmt --all && cd ./enclave && cargo fmt --all
@cd ./enclave && cargo fmt --all
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since cargo fmt failed on the directory.

 $ cargo fmt --all
`cargo metadata` exited with an error: error: could not find `Cargo.toml` in `/home/ubuntu/ghq/github.com/datachainlab/besu-ethereum-ibc-lcp` or any parent directory

setup_runtime!({
Environment::new(build_lc_registry())
});
setup_runtime!({ Environment::new(build_lc_registry()) });
Copy link
Contributor Author

@junichi-tanaka junichi-tanaka Feb 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change is just formatted by make fmt.

Comment on lines -44 to -45
- --allow-insecure-unlock
- --unlock=0xa89F47C6b463f74d87572b058427dA0A13ec5425,0xcBED645B1C1a6254f1149Df51d3591c6B3803007
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines 20 to 26
accounts: {
mnemonic: "math razor capable expose worth grape metal sunset metal sudden usage scheme",
path: "m/44'/60'/0'/0",
initialIndex: 0,
count: 1,
passphrase: "",
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

module.exports = {
solidity: {
version: "0.8.20",
version: "0.8.28",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: try to add cancun to Besu-side as well

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since cancun isn't enabled in genesis.json, it's okay not to enable cancun on Besu-side.
https://github.com/datachainlab/besu-ethereum-ibc-lcp/blob/main/tests/e2e/chains/besu/chain/qbftConfigFile.json

@junichi-tanaka junichi-tanaka changed the title [WIP] Bump lcp to v0.2.12 Bump lcp to v0.2.12 Feb 25, 2025
Comment on lines +45 to +47
},
"minimal_fork_sched": {
"electra": 82125
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@junichi-tanaka junichi-tanaka marked this pull request as ready for review February 25, 2025 17:30
@@ -1,23 +1,21 @@
module github.com/datachainlab/cosmos-ethereum-ibc-lcp
module github.com/datachainlab/besu-ethereum-ibc-lcp
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Member

@bluele bluele left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@junichi-tanaka Sorry for the late review🙇 LGTM👍

@junichi-tanaka junichi-tanaka merged commit ce3ed8d into main Apr 10, 2025
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