Geoffery10 / Discord-Role-Manager / 27890045690

90%

push

github

Remove dead DEBUG branch in check_if_update_needed

check_if_update_needed had a 'DEBUG = False' local variable with an
if/else around the last_update.json write. The 'else' branch was
unreachable — DEBUG was hardcoded False at the top of the block, so
nothing could ever set it to True without source modification.

Inlined the JSON write into the surrounding if-block. The function
now reads as the day-changed gate it was meant to be: if today differs
from last_update.json, write the new date and run update().

If a 'skip the write for debugging' bypass is ever wanted back, it
should be wired through a real env var or function arg, not a local
boolean.

Side effect: update.py coverage jumps from 98% to 100% since the
unreachable line is gone.

2 of 2 new or added lines in 1 file covered. (100.0%)

1473 of 2273 relevant lines covered (64.8%)

0.65 hits per line