Skip to content

Conversation

danielbui12
Copy link
Contributor

No description provided.

tracing = { workspace = true }
blueprint-runner = { workspace = true, features = ["networking", "tangle", "std"] }
blueprint-clients = { workspace = true, features = ["std", "tangle"] }
blueprint-runner = { workspace = true, features = ["networking", "tangle", "eigenlayer", "std"] }
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be behind a feature flag, tangle + eigenlayer doesn't go together.


#[inline]
#[must_use]
pub fn eigen_blueprint_binary_path(&self) -> Option<&Path> {
Copy link
Contributor

Choose a reason for hiding this comment

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

Why would we need this over the existing method blueprint_config_path?

@@ -0,0 +1,130 @@
use crate::config::BlueprintManagerContext;
Copy link
Contributor

Choose a reason for hiding this comment

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

This entire file seems wrong. There is nothing exactly unique about Eigenlayer events. Eigenlayer is a protocol on EVM that uses Solidity smart contracts or otherwise EVM compatible languages to emit events. The EVM event handling utilities should be sufficient for anything here.

use url::Url;

// ================= Core Eigenlayer Deployment Addresses =================
// TODO(daniel): update these addresses to the new addresses of Tangle testnet
Copy link
Contributor

Choose a reason for hiding this comment

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

These have nothing to do with Tangle testnet, we care only about anvil local testnet with forked state, the live testnet eigenlayer uses, and mainnet where eigenlayer is deployed. None of these are Tangle.

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.

2 participants