Fix bug in the ieeetr_rules.check_article rule (#16)

s-heppner · web-flow · commit 2cd8b8bb4815 · 2025-06-22T13:57:08.000+02:00
Previously, the field `number` was considered to be ommitted in the rendered bibliography, which turned out to be a bug, as it is being rendered. This removes the `number` field from the `ommitted_fields` in the linter rule. Fixes #13
diff --git a/bibtex_linter/ieeetr_rules.py b/bibtex_linter/ieeetr_rules.py
@@ -26,7 +26,6 @@ def check_article(entry: BibTeXEntry) -> List[str]:
     }
     omitted_fields: Set[str] = {
         "language",
-        "number",
         "url",
     }
     invariant_violations.extend(check_required_fields(entry, required_fields))

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,6 @@ def check_article(entry: BibTeXEntry) -> List[str]:`
`26`	`26`	`}`
`27`	`27`	`omitted_fields: Set[str] = {`
`28`	`28`	`"language",`
`29`		`- "number",`
`30`	`29`	`"url",`
`31`	`30`	`}`
`32`	`31`	`invariant_violations.extend(check_required_fields(entry, required_fields))`