Consider removing the commented out code block PY-W0069
Anti-pattern
Major
a year ago
—
a year old
Consider removing the commented out code block
359 response = await get_ios_cfw()
360 found_jbs = []
361 jailbreaks = response.get("jailbreak")
362 # jailbreaks = [jb for _, jb in jailbreaks.items()]363 for jb in jailbreaks:
364 if jb.get("compatibility") is None:
365 continueConsider removing the commented out code block
315 "`, `".join(model_numbers) + "`", inline=True)
316
317 ios = response.get("ios")
318 # ios = [i for _, i in ios.items()]319 supported_firmwares = [firmware for firmware in ios if model_number.get(
320 "key") in firmware.get("devices")]
321 supported_firmwares.sort(key=lambda x: x.get("released") or "")Consider removing the commented out code block
203 for os_version in ["iOS", "tvOS", "watchOS", "audioOS"]:
204 version = version.replace(os_version + " ", "")
205 ios = response.get("ios")
206 # ios = [i for _, i in ios.items()]207 ios = [ios for ios in ios if (f"{ios.get('osStr')} {ios.get('version')} ({ios.get('build')})" == og_version or ios.get(
208 'uniqueBuild').lower() == version.lower() or ios.get('version').lower() == version.lower()) and ios.get('beta')]
209Consider removing the commented out code block
179 for os_version in ["iOS", "tvOS", "watchOS", "audioOS"]:
180 version = version.replace(os_version + " ", "")
181 ios = response.get("ios")
182 # ios = [i for _, i in ios.items()]183
184 ios = [ios for ios in ios if f"{ios.get('osStr')} {ios.get('version')} ({ios.get('build')})" == og_version or ios.get(
185 'uniqueBuild').lower() == version.lower() or ios.get('version').lower() == version.lower()]Consider removing the commented out code block
117 # await test.delete(delay=delete_after)
118 # return
119
120 # ephemeral = kwargs.get("ephemeral")121 # if kwargs.get("ephemeral") is not None:
122 # del kwargs["ephemeral"]
123 # delete_after = kwargs.get("delete_after")
Description
It is recommended to remove any commented code in your codebase.
Bad practice
for item in sequence:
# print(item)
do_something(item)
# def old_function():
# '''Older implementation that has been replaced'''
# data = get_data()
# api.post(data)
Recommended
for item in sequence:
do_something(item)