mirrors/ruff
1
0
Fork 0
mirror of https://github.com/astral-sh/ruff.git synced 2025-09-29 05:14:52 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

[Ruff v0.5] Fix ZeroDivisionErrors in the ecosystem check (#12027)

Browse source 
Seen in CI in https://github.com/astral-sh/ruff/pull/12026
This commit is contained in:
Alex Waygood 2024-06-25 13:48:25 +01:00 • committed by Micha Reiser
parent a4d711f25f
commit 0a24d70bfd
1 changed files with 8 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

9
python/ruff-ecosystem/ruff_ecosystem/check.py
View file

@ -153,6 +153,9 @@ def markdown_check_result(result: Result) -> str:
# skip display. This shouldn't really happen and indicates a problem in the # skip display. This shouldn't really happen and indicates a problem in the
# calculation of these values. Instead of skipping entirely when `total_changes` # calculation of these values. Instead of skipping entirely when `total_changes`
# is zero, we'll attempt to report the results to help diagnose the problem. # is zero, we'll attempt to report the results to help diagnose the problem.
#
# There's similar issues with the `max_display_per_rule` calculation immediately
# below as well.
project_changes / max(total_changes, 1) project_changes / max(total_changes, 1)
) )
* 50 * 50
@ -162,7 +165,11 @@ def markdown_check_result(result: Result) -> str:
# Limit the number of items displayed per rule to between 5 and the max for # Limit the number of items displayed per rule to between 5 and the max for
# the project based on the number of rules affected (less rules, more per rule) # the project based on the number of rules affected (less rules, more per rule)
max_display_per_rule = max( max_display_per_rule = max(
5, max_display_per_project // len(rule_changes.rule_codes()) 5,
# TODO: remove the need for the max() call here,
# which is a workaround for if `len(rule_changes.rule_codes()) == 0`
# (see comment in the assignment of `max_display_per_project` immediately above)
max_display_per_project // max(len(rule_changes.rule_codes()), 1),
) )
# Display the diff # Display the diff