Make agave-unstable-api default for votor/ so we don't need to specify it many times. (anza-xyz#8257)

* Make agave-unstable-api default for votor/ so we don't need to specify it many times.

* Use default = [ "agave-unstable-api" ] in Cargo.toml instead, this makes `cargo test` work.

* Make linter happy.

* Should still put everything under agave-unstable-api.

Author: wen-coding

votor/Cargo.toml

@@ -11,6 +11,7 @@ edition = { workspace = true }
 
 [features]
 agave-unstable-api = []
+default = ["agave-unstable-api"]
 dev-context-only-utils = ["solana-runtime/dev-context-only-utils"]
 frozen-abi = [
     "dep:solana-frozen-abi",

votor/src/lib.rs

@@ -1,45 +1,21 @@
+#![cfg(feature = "agave-unstable-api")]
 #![cfg_attr(feature = "frozen-abi", feature(min_specialization))]
 
-#[cfg(feature = "agave-unstable-api")]
-pub mod commitment;
+#[macro_use]
+extern crate log;
+extern crate serde_derive;
 
-#[cfg(feature = "agave-unstable-api")]
+pub mod commitment;
 pub mod common;
-
-#[cfg(feature = "agave-unstable-api")]
 mod consensus_metrics;
-
-#[cfg(feature = "agave-unstable-api")]
 pub mod consensus_pool;
-
-#[cfg(feature = "agave-unstable-api")]
 pub mod event;
-
-#[cfg(feature = "agave-unstable-api")]
 pub mod root_utils;
-
-#[cfg(feature = "agave-unstable-api")]
-#[macro_use]
-extern crate log;
-
-#[cfg(feature = "agave-unstable-api")]
-extern crate serde_derive;
-
-#[cfg(feature = "agave-unstable-api")]
 mod staked_validators_cache;
-
-#[cfg(feature = "agave-unstable-api")]
 mod timer_manager;
-
-#[cfg(feature = "agave-unstable-api")]
 pub mod vote_history;
-#[cfg(feature = "agave-unstable-api")]
 pub mod vote_history_storage;
-
-#[cfg(feature = "agave-unstable-api")]
 mod voting_service;
-
-#[cfg(feature = "agave-unstable-api")]
 mod voting_utils;
 
 #[cfg_attr(feature = "frozen-abi", macro_use)]