Warn on unreachable_pub lint

This makes it easier to track visibility/compatibility during review.

src/connection.rs

@@ -16,10 +16,10 @@ use crate::error::Error;
 
 /// EPP Connection struct with some metadata for the connection
 pub(crate) struct EppConnection<C: Connector> {
-    pub registry: String,
+    pub(crate) registry: String,
     connector: C,
     stream: C::Connection,
-    pub greeting: String,
+    pub(crate) greeting: String,
     timeout: Duration,
     // A request that is currently in flight
     //

src/domain/mod.rs

@@ -82,11 +82,11 @@ fn deserialize_host_addrs_option<'xml>(
 /// The `<hostAddr>` types domain or host transactions
 #[derive(Debug, FromXml, ToXml)]
 #[xml(rename = "hostAddr", ns(super::domain::XMLNS))]
-pub(crate) struct HostAddr<'a> {
+struct HostAddr<'a> {
     #[xml(attribute, rename = "ip")]
-    pub ip_version: Option<Cow<'a, str>>,
+    ip_version: Option<Cow<'a, str>>,
     #[xml(direct)]
-    pub address: Cow<'a, str>,
+    address: Cow<'a, str>,
 }
 
 impl From<&IpAddr> for HostAddr<'static> {

src/hello.rs

@@ -24,12 +24,12 @@ pub struct ServiceMenu {
 #[derive(Debug, FromXml, PartialEq)]
 #[xml(ns(EPP_XMLNS), rename = "svcMenu")]
 struct FlattenedServiceMenu {
-    pub version: String,
-    pub lang: String,
+    version: String,
+    lang: String,
     #[xml(rename = "objURI")]
-    pub obj_uris: Vec<String>,
+    obj_uris: Vec<String>,
     #[xml(rename = "svcExtension")]
-    pub svc_ext: Option<ServiceExtension<'static>>,
+    svc_ext: Option<ServiceExtension<'static>>,
 }
 
 impl<'xml> FromXml<'xml> for ServiceMenu {

src/host/mod.rs

@@ -128,11 +128,11 @@ impl<'xml> FromXml<'xml> for Status {
 /// The `<hostAddr>` types domain or host transactions
 #[derive(Debug, FromXml, ToXml)]
 #[xml(rename = "addr", ns(XMLNS))]
-pub(crate) struct HostAddr<'a> {
+struct HostAddr<'a> {
     #[xml(attribute, rename = "ip")]
-    pub ip_version: Option<Cow<'a, str>>,
+    ip_version: Option<Cow<'a, str>>,
     #[xml(direct)]
-    pub address: Cow<'a, str>,
+    address: Cow<'a, str>,
 }
 
 impl From<&IpAddr> for HostAddr<'static> {

src/lib.rs

@@ -32,6 +32,8 @@
 //! You will usually want to start by initializing an [`EppClient`]. Refer to the example code
 //! on that type for more information.
 
+#![warn(unreachable_pub)]
+
 pub mod client;
 pub mod common;
 pub mod connection;

src/request.rs

@@ -25,12 +25,12 @@ pub trait Extension: ToXml + Debug {
 /// Type corresponding to the `<command>` tag in an EPP XML request
 /// with an `<extension>` tag
 pub(crate) struct CommandWrapper<'a, D, E> {
-    pub command: &'static str,
+    command: &'static str,
     /// The instance that will be used to populate the `<command>` tag
-    pub data: &'a D,
+    data: &'a D,
     /// The client TRID
-    pub extension: Option<&'a E>,
-    pub client_tr_id: String,
+    extension: Option<&'a E>,
+    client_tr_id: String,
 }
 
 impl<'a, E: Extension, D: Transaction<E>> CommandWrapper<'a, D, E> {