new lint: empty_inherit

1 files changed, 53 insertions, 0 deletions

diff --git a/lib/src/lints/empty_inherit.rs b/lib/src/lints/empty_inherit.rs
new file mode 100644
index 0000000..9ae4cf2
--- /dev/null
+++ b/lib/src/lints/empty_inherit.rs
@@ -0,0 +1,53 @@
+use crate::{make, utils, Metadata, Report, Rule, Suggestion};
+
+use if_chain::if_chain;
+use macros::lint;
+use rnix::{
+    types::{Inherit, TypedNode},
+    NodeOrToken, SyntaxElement, SyntaxKind,
+};
+
+/// ## What it does
+/// Checks for empty inherit statements.
+///
+/// ## Why is this bad?
+/// Useless code, probably the result of a refactor.
+///
+/// ## Example
+///
+/// ```nix
+/// inherit;
+/// ```
+///
+/// Remove it altogether.
+#[lint(
+    name = "empty_inherit",
+    note = "Found empty inherit statement",
+    code = 14,
+    match_with = SyntaxKind::NODE_INHERIT
+)]
+struct EmptyInherit;
+
+impl Rule for EmptyInherit {
+    fn validate(&self, node: &SyntaxElement) -> Option<Report> {
+        if_chain! {
+            if let NodeOrToken::Node(node) = node;
+            if let Some(inherit_stmt) = Inherit::cast(node.clone());
+            if inherit_stmt.from().is_none();
+            if inherit_stmt.idents().count() == 0;
+            then {
+                let at = node.text_range();
+                let replacement = make::empty().node().clone();
+                let replacement_at = utils::with_preceeding_whitespace(node);
+                let message = "Remove this empty `inherit` statement";
+                Some(
+                    self
+                    .report()
+                    .suggest(at, message, Suggestion::new(replacement_at, replacement))
+                )
+            } else {
+                None
+            }
+        }
+    }
+}