Move deny missing_docs to ci-doc.sh

Author: qinsoon

.github/scripts/ci-doc.sh

@@ -4,7 +4,7 @@
 # If the output path is changed in this script, we need to update rustdoc.yml as well.
 
 # deny warnings for rustdoc
-export RUSTDOCFLAGS="-D warnings"
+export RUSTDOCFLAGS="-D warnings -D missing_docs"
 
 # Check cargo doc
 # We document public and private items for MMTk developers (GC implementers).

src/lib.rs

@@ -1,6 +1,3 @@
-// Deny missing_docs: if any public item does not have a proper rustdoc comment, the build will fail.
-// We try to keep the code well documented, especially for items in our public API.
-#![deny(missing_docs)]
 // Allow this for now. Clippy suggests we should use Sft, Mmtk, rather than SFT and MMTK.
 // According to its documentation (https://rust-lang.github.io/rust-clippy/master/index.html#upper_case_acronyms),
 // with upper-case-acronyms-aggressive turned on, it should also warn us about SFTMap, VMBinding, GCWorker.