IonStromRework

[TryHardo7]

Переработал систему, генерирующую бессмысленную белеберду в законы при ионных штормах по типу той, что на скриншотах, теперь в качестве ионок будут вразумительные законы, побуждающие забавный и/или интересный игровой опыт. На данный момент вариантов ионных законов 42. Список ионных законов всегда можно будет дополнять или сокращать.

• PR полностью завершён и мне не нужна помощь чтобы его закончить.
• Я ознакомился с наставлениями по работе с репозиторием и следовал им при создании PR'а.
• Я внимательно просмотрел все свои изменения и багов в них не нашёл.
• Я запускал локальный сервер со своими изменениями и всё протестировал.
• Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

🆑 Lancevrot

• tweak: Изменена система ионных законов

[Ady4ik]

не надо строчки было удалять.. Мы при апстриме будем выть, верните пожалуйста и закоментите...

[lexaSvarshik]

Небольшие исправления локализации

[Kirus59]

Ради всего святого только код, 1000 строчек рандомных слов то можно потереть?

Нет конечно, я не понимаю зачем ты вообще полез в тот файл, а тем более удалил/закомментил константы и почти весь метод GenerateLaws.

Лучшим решением будет создать отдельный метод для генерации законов так как ты хотел и вызывать его вместо GenerateLaws + создать датасет в НАШЕЙ папке.

[Kirus59]

А тем более что отдельный метод тебе судя по всему и не нужен, а просто рандомный пик закона из датасета.
Поэтому можно просто заккоментить этот вар и вместо него написать свой рандом пик

[TryHardo7]

А тем более что отдельный метод тебе судя по всему и не нужен, а просто рандомный пик закона из датасета. Поэтому можно просто заккоментить этот вар и вместо него написать свой рандом пик

Ну частично закоментить метод, который был придётся в любом случае, он же конфликтовать будет, при условии что я вернул оригинальный датасет. Новый датасет сделал в папке сс220, оставил генерацию как есть, там 3 строчки кода, вроде эффективно. Можно их тоже конечно закоментить и написать рядом буквально то-же самое? По другому я не знаю как сделать

[Kirus59]

Ну закоментить метод, который был придётся в любом случае, он же конфликтовать будет, при условии что я вернул оригинальный датасет. Новый датасет сделал в папке сс220, оставил генерацию как есть, там 3 строчки кода, вроде эффективно. Можно их тоже конечно закоментить и написать рядом буквально то-же самое? По другому я не знаю как сделать

Что конфликтовать будет?
Я тебе говорю - просто замени вызов этого метода на рандом пик из датасета, не понимаю в чем конфликт.
То же самое касается и датасетов. с чем они будут конфликтовать если используются только внутри метода?
Если ты не вызываешь этот метод - соответственно и не происходит генерация закона.

[Kirus59]

Если ты про мерж конфликты, то от закомменчивания всех констант и целого метода будет гораздо больше конфликтов, чем от замены одной переменной.

[TryHardo7]

Ну вроде по человечески сделал, работает.

[UrPrice]

Все названия энтити должны быть с маленькой буквы. Все описания должны быть с большой буквы.

[TryHardo7]

Названия энтити такие, какие они в коде. А описания и есть с заглавной буквы

Либо я вообще не понял о чём ты говоришь

[Kirus59]

Названия энтити такие, какие они в коде. А описания и есть с заглавной буквы

Либо я вообще не понял о чём ты говоришь

Названия энтити принято писать с маленькой буквы (кроме имен кукол и животных если они есть).
Если где-то еще видел с большой - то вероятнее всего ошибка.

На скрине Урист скорее относится к имени куклы

[Kirus59]

Ну и думаю что пока не стоит трогать переводы оффовских плат, т.к. потом все равно подтянем перевод от Корвакса

[UrPrice]

Урист МакВульпа это имя собственное. А нарицательные остаются с маленькой.

[TryHardo7]

Дело шло бы лучше, если бы я хоть что-то понимал в коде.

Так файл локализации плат убрать?

[Kirus59]

Дело шло бы лучше, если бы я хоть что-то понимал в коде.

Так файл локализации плат убрать?

Да, у корвакса они уже переведены, так что вскоре черри-пикнем и нам

[Kirus59]

В остальном норм, по законам - обсудим с Адичем

[github-actions]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

[TryHardo7]

Пока мы тут сидели, залетела локализация с корвакса?

[Kirus59]

Пока мы тут сидели, залетела локализация с корвакса?

Я как раз вчера черри-пик и делал

[TryHardo7]

Убрал локализацию, остался только реворк ионных законов.

Почему у меня гит каждый день требует подтверждение почты?

[TryHardo7]

Он проваливает чек с поджигаемостью стены
Либо я что-то не понимаю, либо это пришло из мастер ветки

[lexaSvarshik]

Он проваливает чек с поджигаемостью стены Либо я что-то не понимаю, либо это пришло из мастер ветки

У меня такое тоже было, мерджни мастер в ветку

[Kirus59]

Технически все норм, жду апрува от Адича

[Ady4ik]

Вроде прикольно, согласую с сурксом, чтобы проблем со стороны правил не было

[UrPrice]

Мёрджу в связи с выпуском ИИ на все станции. Суркс и Адуч уже потом дадут комментарии и если что дополнительным патчем прольём фикс, если он вообще потребуется.