Add small module-level docs

diff --git a/crates/ra_hir/src/from_id.rs b/crates/ra_hir/src/from_id.rs
index c08203bca7ea236bb05ae77d7acaa24449f29e99..089dbc9084a454f09248d3f229e1366099f95a85 100644 (file)
--- a/crates/ra_hir/src/from_id.rs
+++ b/crates/ra_hir/src/from_id.rs
@@ -1,3 +1,8 @@
+//! Utility module for converting between hir_def ids and code_model wrappers.
+//!
+//! It's unclear if we need this long-term, but it's definitelly useful while we
+//! are splitting the hir.
+
 use hir_def::{AdtId, EnumVariantId, ModuleDefId};
 
 use crate::{Adt, EnumVariant, ModuleDef};

diff --git a/crates/ra_hir_def/src/diagnostics.rs b/crates/ra_hir_def/src/diagnostics.rs
index 637184c58229b2eb3aeb2806090d2854ee234829..1c1ccdecb72f69dbb92878fba20d8bc3773baac8 100644 (file)
--- a/crates/ra_hir_def/src/diagnostics.rs
+++ b/crates/ra_hir_def/src/diagnostics.rs
@@ -1,3 +1,5 @@
+//! Diagnostics produced by `hir_def`.
+
 use std::any::Any;
 
 use hir_expand::diagnostics::Diagnostic;

diff --git a/crates/ra_hir_def/src/test_db.rs b/crates/ra_hir_def/src/test_db.rs
index 05018f8e43c2e06baf425826b17e669691292c55..f91a5b41df2bb434a777c439a7ee3845acf193b5 100644 (file)
--- a/crates/ra_hir_def/src/test_db.rs
+++ b/crates/ra_hir_def/src/test_db.rs
@@ -1,3 +1,5 @@
+//! Database used for testing `hir_def`.
+
 use std::{
     panic,
     sync::{Arc, Mutex},

diff --git a/xtask/tests/tidy-tests/docs.rs b/xtask/tests/tidy-tests/docs.rs
index 6a629ce63e8fc6cec66ee3d7d9408f3d9607d0eb..227937f462a626641beb929fe4402270f5c5f2f5 100644 (file)
--- a/xtask/tests/tidy-tests/docs.rs
+++ b/xtask/tests/tidy-tests/docs.rs
@@ -36,6 +36,7 @@ fn is_hidden(entry: &DirEntry) -> bool {
 fn no_docs_comments() {
     let crates = project_root().join("crates");
     let iter = WalkDir::new(crates);
+    let mut missing_docs = Vec::new();
     for f in iter.into_iter().filter_entry(|e| !is_hidden(e)) {
         let f = f.unwrap();
         if f.file_type().is_dir() {
@@ -54,12 +55,14 @@ fn no_docs_comments() {
         let mut line = String::new();
         reader.read_line(&mut line).unwrap();
         if !line.starts_with("//!") {
-            panic!(
-                "\nMissing docs strings\n\
-                 module: {}\n\
-                 Need add doc for module\n",
-                f.path().display()
-            )
+            missing_docs.push(f.path().display().to_string());
         }
     }
+    if !missing_docs.is_empty() {
+        panic!(
+            "\nMissing docs strings\n\n\
+             modules:\n{}\n\n",
+            missing_docs.join("\n")
+        )
+    }
 }