Enforce doc link correctness during cargo doc runs and in CI

Author: TheBlueMatt

background-processor/src/lib.rs

@@ -1,3 +1,6 @@
+#![deny(broken_intra_doc_links)]
+#![deny(unsafe_code)]
+
 #[macro_use] extern crate lightning;
 
 use lightning::chain;

ci/check-compiles.sh

@@ -3,4 +3,5 @@ set -e
 set -x
 echo Testing $(git log -1 --oneline)
 cargo check
+cargo doc
 cd fuzz && cargo check --features=stdin_fuzz

lightning-block-sync/src/lib.rs

@@ -16,6 +16,9 @@
 //! [`SpvClient`]: struct.SpvClient.html
 //! [`BlockSource`]: trait.BlockSource.html
 
+#![deny(broken_intra_doc_links)]
+#![deny(unsafe_code)]
+
 #[cfg(any(feature = "rest-client", feature = "rpc-client"))]
 pub mod http;
 

lightning-net-tokio/src/lib.rs

@@ -72,6 +72,9 @@
 //! }
 //! ```
 
+#![deny(broken_intra_doc_links)]
+#![deny(missing_docs)]
+
 use bitcoin::secp256k1::key::PublicKey;
 
 use tokio::net::TcpStream;

lightning-persister/src/lib.rs

@@ -1,3 +1,5 @@
+#![deny(broken_intra_doc_links)]
+
 mod util;
 
 extern crate lightning;

lightning/src/lib.rs

@@ -20,6 +20,7 @@
 
 #![cfg_attr(not(any(feature = "fuzztarget", feature = "_test_utils")), deny(missing_docs))]
 #![cfg_attr(not(any(test, feature = "fuzztarget", feature = "_test_utils")), forbid(unsafe_code))]
+#![deny(broken_intra_doc_links)]
 
 // In general, rust is absolutely horrid at supporting users doing things like,
 // for example, compiling Rust code for real environments. Disable useless lints