code review

crates/rome_formatter/src/formatter.rs

@@ -266,7 +266,13 @@ impl Formatter {
         Ok(result.into_iter())
     }
 
-    /// It formats a list of nodes that are not separated.
+    /// It formats a list of nodes that are not separated. It's a ad-hoc function to
+    /// format lists that implement [rslint_parser::AstNodeList].
+    ///
+    /// The elements of the list are joined together using [join_elements_hard_line], which will
+    /// end up separated by hard lines or empty lines.
+    ///
+    /// If the formatter fails to format an element, said element gets printed verbatim.
     pub fn format_list<List, Node: Clone + AstNode + ToFormatElement>(
         &self,
         list: List,

crates/rome_formatter/src/ts/root/module.rs

@@ -1,7 +1,8 @@
 use crate::ts::directives::format_directives_list;
 use crate::ts::root::format_interpreter;
 use crate::{
-    format_elements, hard_line_break, FormatElement, FormatResult, Formatter, ToFormatElement,
+    format_elements, hard_line_break, join_elements_hard_line, FormatElement, FormatResult,
+    Formatter, ToFormatElement,
 };
 use rslint_parser::ast::JsModule;
 

crates/rome_formatter/tests/specs/js/module/newlines.js.snap

@@ -1,6 +1,6 @@
 ---
 source: crates/rome_formatter/tests/spec_test.rs
-assertion_line: 43
+assertion_line: 57
 expression: newlines.js
 
 ---