From 08d5bc6d6bcda8dbc98c0e8faa3e5876d4eab367 Mon Sep 17 00:00:00 2001
From: Paul Hauner <paul@paulhauner.com>
Date: Sat, 18 Feb 2023 16:28:36 +1100
Subject: [PATCH] Remove `FreeAttestation`

This is an ancient relic, I'm surprised it still existed!
---
 consensus/types/src/free_attestation.rs | 13 -------------
 consensus/types/src/lib.rs              |  2 --
 2 files changed, 15 deletions(-)
 delete mode 100644 consensus/types/src/free_attestation.rs

diff --git a/consensus/types/src/free_attestation.rs b/consensus/types/src/free_attestation.rs
deleted file mode 100644
index dd3782d3ce..0000000000
--- a/consensus/types/src/free_attestation.rs
+++ /dev/null
@@ -1,13 +0,0 @@
-/// Note: this object does not actually exist in the spec.
-///
-/// We use it for managing attestations that have not been aggregated.
-use super::{AttestationData, Signature};
-use serde_derive::Serialize;
-
-#[derive(arbitrary::Arbitrary, Debug, Clone, PartialEq, Serialize)]
-pub struct FreeAttestation {
-    pub data: AttestationData,
-    pub signature: Signature,
-    #[serde(with = "eth2_serde_utils::quoted_u64")]
-    pub validator_index: u64,
-}
diff --git a/consensus/types/src/lib.rs b/consensus/types/src/lib.rs
index 8d9156ff5d..2926a434b1 100644
--- a/consensus/types/src/lib.rs
+++ b/consensus/types/src/lib.rs
@@ -47,7 +47,6 @@ pub mod fork;
 pub mod fork_data;
 pub mod fork_name;
 pub mod fork_versioned_response;
-pub mod free_attestation;
 pub mod graffiti;
 pub mod historical_batch;
 pub mod historical_summary;
@@ -154,7 +153,6 @@ pub use crate::fork_name::{ForkName, InconsistentFork};
 pub use crate::fork_versioned_response::{
     ExecutionOptimisticForkVersionedResponse, ForkVersionDeserialize, ForkVersionedResponse,
 };
-pub use crate::free_attestation::FreeAttestation;
 pub use crate::graffiti::{Graffiti, GRAFFITI_BYTES_LEN};
 pub use crate::historical_batch::HistoricalBatch;
 pub use crate::indexed_attestation::IndexedAttestation;