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

Cleanup tx's result.deployments #666

Open
Karrq opened this issue Oct 28, 2024 · 0 comments · Fixed by #681
Open

Cleanup tx's result.deployments #666

Karrq opened this issue Oct 28, 2024 · 0 comments · Fixed by #681
Labels
good first issue 🐣 Good for newcomers needs triage ♟️ Issue needs to be picked up or assigned p3 🔵 Indicates low priority item

Comments

@Karrq
Copy link
Contributor

Karrq commented Oct 28, 2024

At some point in time we added this concept of “deployments”, which I believe was necessary because we couldn’t detect the deployments anymore, but now we have proper traces, so we should be able to clean this up.

let mut zk_setup_deployments = HashMap::new();

@Karrq Karrq added good first issue 🐣 Good for newcomers p3 🔵 Indicates low priority item needs triage ♟️ Issue needs to be picked up or assigned labels Oct 28, 2024
@Karrq Karrq linked a pull request Oct 31, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue 🐣 Good for newcomers needs triage ♟️ Issue needs to be picked up or assigned p3 🔵 Indicates low priority item
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant