Remove unnecessary pub(crate) and bump versions of consensus and rmc

consensus/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "aleph-bft"
-version = "0.23.0"
+version = "0.24.0"
 edition = "2021"
 authors = ["Cardinal Cryptography"]
 categories = ["algorithms", "data-structures", "cryptography", "database"]
@@ -13,7 +13,7 @@ readme = "../README.md"
 description = "AlephBFT is an asynchronous and Byzantine fault tolerant consensus protocol aimed at ordering arbitrary messages (transactions). It has been designed to continuously operate even in the harshest conditions: with no bounds on message-delivery delays and in the presence of malicious actors. This makes it an excellent fit for blockchain-related applications."
 
 [dependencies]
-aleph-bft-rmc = { path = "../rmc", version = "0.7" }
+aleph-bft-rmc = { path = "../rmc", version = "0.8" }
 aleph-bft-types = { path = "../types", version = "0.8" }
 anyhow = "1.0"
 async-trait = "0.1"

consensus/src/alerts/mod.rs

@@ -80,7 +80,7 @@ impl<H: Hasher, D: Data, S: Signature> Alert<H, D, S> {
         self.proof.0.as_signable().creator()
     }
 
-    pub(crate) fn included_data(&self) -> Vec<D> {
+    pub fn included_data(&self) -> Vec<D> {
         // Only legit units might end up in the DAG, we can ignore the fork proof.
         self.legit_units
             .iter()
@@ -114,7 +114,7 @@ pub enum AlertMessage<H: Hasher, D: Data, S: Signature, MS: PartialMultisignatur
 }
 
 impl<H: Hasher, D: Data, S: Signature, MS: PartialMultisignature> AlertMessage<H, D, S, MS> {
-    pub(crate) fn included_data(&self) -> Vec<D> {
+    pub fn included_data(&self) -> Vec<D> {
         match self {
             Self::ForkAlert(unchecked_alert) => unchecked_alert.as_signable().included_data(),
             Self::RmcMessage(_, _) => Vec::new(),

rmc/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "aleph-bft-rmc"
-version = "0.7.0"
+version = "0.8.0"
 edition = "2021"
 authors = ["Cardinal Cryptography"]
 categories = ["algorithms", "cryptography"]