Skip to content

Commit b0ec952

Browse files
committed
fix: sort binary checkers
Sort binary checkers for readability as already done for language checkers Signed-off-by: Fabrice Fontaine <[email protected]>
1 parent 8f8acb3 commit b0ec952

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

cve_bin_tool/version_scanner.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,9 @@ def __init__(
7070
self.error_mode = error_mode
7171
self.cve_db = CVEDB(sources=sources)
7272
self.validate = validate
73-
self.logger.info("Checkers loaded: %s" % (", ".join(self.checkers.keys())))
73+
self.logger.info(
74+
"Checkers loaded: %s" % (", ".join(sorted(self.checkers.keys())))
75+
)
7476
self.language_checkers = valid_files
7577
self.language_checkers_names = self.available_language_checkers()
7678

@@ -102,7 +104,7 @@ def available_checkers(cls) -> list[str]:
102104
)
103105
]
104106
all_checker_names = list(BUILTIN_CHECKERS.keys()) + entrypoint_checker_names
105-
return all_checker_names
107+
return sorted(all_checker_names)
106108

107109
def remove_skiplist(self, skips: list[str]) -> None:
108110
"""Remove specific checkers If a checker is in the skip list, it will be removed from this dictionary. If it's not found or is not a valid checker name, it logs error messages."""
@@ -120,7 +122,7 @@ def remove_skiplist(self, skips: list[str]) -> None:
120122
def print_checkers(self) -> None:
121123
"""Print checkers and logs an informational message that lists the names of the checkers stored in the dictionary"""
122124

123-
self.logger.info(f'Checkers: {", ".join(self.checkers.keys())}')
125+
self.logger.info(f'Checkers: {", ".join(sorted(self.checkers.keys()))}')
124126

125127
def number_of_checkers(self) -> int:
126128
"""return the number of checkers"""

0 commit comments

Comments
 (0)