Merge pull request #445 from Automattic/nor-oxford-comma

fix(core): the Oxford comma applies to `nor` as well

harper-core/src/linting/oxford_comma.rs

@@ -21,7 +21,7 @@ impl OxfordComma {
                 ))
                 .then_noun_phrase()
                 .then_whitespace()
-                .then(Box::new(WordSet::all(&["and", "or"])))
+                .then(Box::new(WordSet::all(&["and", "or", "nor"])))
                 .then_whitespace()
                 .then_noun_phrase(),
         }
@@ -81,7 +81,7 @@ impl Linter for OxfordComma {
     }
 
     fn description(&self) -> &str {
-        "The Oxford comma is one of the more controversial rules in common use today. Enabling this lint checks that there is a comma before `and` or `or` when listing out more than two ideas."
+        "The Oxford comma is one of the more controversial rules in common use today. Enabling this lint checks that there is a comma before `and`, `or`, or `nor` when listing out more than two ideas."
     }
 }
 
@@ -150,4 +150,13 @@ mod tests {
             "They enjoy playing soccer, basketball, or tennis.",
         );
     }
+
+    #[test]
+    fn nor_vegetables() {
+        assert_suggestion_result(
+            "I like carrots, kale nor broccoli.",
+            OxfordComma::default(),
+            "I like carrots, kale, nor broccoli.",
+        );
+    }
 }