From: Florian Diebold Date: Sat, 22 May 2021 14:39:19 +0000 (+0200) Subject: Add last remaining module docstring X-Git-Url: https://git.lizzy.rs/?a=commitdiff_plain;h=a88c7c04d28d6263d7bca69ee3014e42185030de;p=rust.git Add last remaining module docstring --- diff --git a/crates/hir_ty/src/diagnostics/expr.rs b/crates/hir_ty/src/diagnostics/expr.rs index b9a69b79c81..53c4ee9da9a 100644 --- a/crates/hir_ty/src/diagnostics/expr.rs +++ b/crates/hir_ty/src/diagnostics/expr.rs @@ -1,4 +1,6 @@ -//! FIXME: write short doc here +//! Various diagnostics for expressions that are collected together in one pass +//! through the body using inference results: mismatched arg counts, missing +//! fields, etc. use std::sync::Arc; diff --git a/xtask/src/tidy.rs b/xtask/src/tidy.rs index f97e751c58a..82b33a7a09c 100644 --- a/xtask/src/tidy.rs +++ b/xtask/src/tidy.rs @@ -1,7 +1,4 @@ -use std::{ - collections::HashMap, - path::{Path, PathBuf}, -}; +use std::path::{Path, PathBuf}; use xshell::{cmd, pushd, pushenv, read_file}; @@ -392,25 +389,9 @@ fn finish(self) { ) } - let poorly_documented = ["hir_ty"]; - - let mut has_fixmes = - poorly_documented.iter().map(|it| (*it, false)).collect::>(); - 'outer: for path in self.contains_fixme { - for krate in poorly_documented.iter() { - if path.components().any(|it| it.as_os_str() == *krate) { - has_fixmes.insert(krate, true); - continue 'outer; - } - } + for path in self.contains_fixme { panic!("FIXME doc in a fully-documented crate: {}", path.display()) } - - for (krate, has_fixme) in has_fixmes.iter() { - if !has_fixme { - panic!("crate {} is fully documented :tada:, remove it from the list of poorly documented crates", krate) - } - } } }