lint: forbid trailing whitespace within <p>

tc39 · Aug 18, 2020 · 6c7e250 · 6c7e250
1 parent 814c4c9
commit 6c7e250
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 0 deletions.
diff --git a/src/lint/collect-spelling-diagnostics.ts b/src/lint/collect-spelling-diagnostics.ts
@@ -65,6 +65,10 @@ let matchers = [
     message:
       'clauses should be referenced using <emu-xref> tags rather than hardcoding clause numbers',
   },
+  {
+    pattern: /(?<=[^ \n]) +<\/p>/gu,
+    message: '<p> tags should not contain trailing whitespace',
+  },
 ];
 
 export function collectSpellingDiagnostics(

diff --git a/test/lint-spelling.js b/test/lint-spelling.js
@@ -211,6 +211,19 @@ windows:${M}\r
     );
   });
 
+  it('<p> trailing whitespace', async function () {
+    await assertLint(
+      positioned`
+        <p>This is an example:${M} </p>
+      `,
+      {
+        ruleId: 'spelling',
+        nodeType: 'html',
+        message: '<p> tags should not contain trailing whitespace',
+      }
+    );
+  });
+
   it('negative', async function () {
     await assertLintFree(`
       <p>
@@ -240,6 +253,10 @@ windows:${M}\r
       </tr></table>
 
       <p>See clause <emu-xref href="#example"></emu-xref>.</p>
+
+      <p>
+        Paragraphs with the open/close tags on their own line are OK.
+      </p>
     `);
   });
 });