Skip to content

Commit 422e29b

Browse files
committed
coverage: Sort modules
This prevent the table changing every time we generate the report. Signed-off-by: Stephen Finucane <[email protected]>
1 parent 434996f commit 422e29b

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

sphinx/ext/coverage.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,7 @@ def build_py_coverage(self) -> None:
247247
', '.join(additional_modules),
248248
)
249249

250-
for mod_name in modules:
250+
for mod_name in sorted(modules):
251251
ignore = False
252252
for exp in self.mod_ignorexps:
253253
if exp.match(mod_name):
@@ -346,8 +346,7 @@ def build_py_coverage(self) -> None:
346346

347347
def _write_py_statistics(self, op: TextIO) -> None:
348348
"""Outputs the table of ``op``."""
349-
all_modules = set(self.py_documented.keys()).union(
350-
set(self.py_undocumented.keys()))
349+
all_modules = frozenset(self.py_documented.keys() | self.py_undocumented.keys())
351350
all_objects: set[str] = set()
352351
all_documented_objects: set[str] = set()
353352
for module in all_modules:
@@ -357,7 +356,7 @@ def _write_py_statistics(self, op: TextIO) -> None:
357356

358357
# prepare tabular
359358
table = [['Module', 'Coverage', 'Undocumented']]
360-
for module in all_modules:
359+
for module in sorted(all_modules):
361360
module_objects = self.py_documented[module].union(self.py_undocumented[module])
362361
if len(module_objects):
363362
value = 100.0 * len(self.py_documented[module]) / len(module_objects)

0 commit comments

Comments
 (0)