Just-Some-Bots / MusicBot

:musical_note: The original MusicBot for Discord (formerly SexualRhinoceros/MusicBot)
https://just-some-bots.github.io/MusicBot/
MIT License
3.14k stars 2.36k forks source link

Replaced string.format () and % with f-strings. #2398

Closed chriszaro closed 6 months ago

chriszaro commented 6 months ago

Pylint reports "[consider-using-f-string". f-string is faster, more readable and futureproof. Pylint now reports about 300 less conventions. There are more to be refactored.

image

After creating your pull request, tick these boxes if they are applicable to you.


Description

Related issues (if applicable)

BabyBoySnow commented 6 months ago

Hello, actually the dev branch is already py linted. We're just holding off on putting it on master for the reason that sev currently uses the most up-to-date discord.py, which hasn't been released on pypi yet so we have changes that won't work. Additionally, please review our pull request guidelines, you've targeted the wrong branch, pull request should target the dev branch. Additionally you haven't said if you've tested these changes.