search

SerVB / TheRecentStatOfYou

Недавняя статистика танкистов / WoT players recent statistics
Apache License 2.0
9 stars 8 forks source link

Create mod_recent_stat.py #1

Closed Mododejl closed 6 years ago

SerVB commented 6 years ago

Благодарю за вклад! Логику подмены методов я понял. А чем мотивировано объединение всего кода в один файл?

Mododejl commented 6 years ago

так надо и так правильно

SerVB commented 6 years ago

Ну, насчёт "так надо" непонятно. Хотелось бы осознать)

"Так правильно" – в модах? Потому что вообще в жизни код принято разделять на файлы по смыслу...

Mododejl commented 6 years ago

ну у мододелов принято все в одном делать и разделять только на классы, если у тебя мод как Battle Observer у которого большой объем кода тогда есть смысл разделять и для модов своя папка есть scripts\client\gui\mods, не нужно городить костыли затрагивая папки скриптов клиента и темболее в них лезть

SerVB commented 6 years ago

Отклоняю PR, потому что хочу иметь многофайловый мод. Тем не менее, с помощью этого PR я понял, как сделать например https://github.com/SerVB/TheRecentStatOfYou/blob/release3/src/scripts/client/gui/mods/mod_recent_stat_loader.py. Благодарю за вклад!