Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Приколы атмоса #329

Merged
merged 5 commits into from
Aug 23, 2024
Merged

Приколы атмоса #329

merged 5 commits into from
Aug 23, 2024

Conversation

Flopochka
Copy link
Contributor

@Flopochka Flopochka commented Aug 21, 2024 •
edited
Loading

Описание PR

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

Проверки

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

Изменения

🆑

  • add: Крафт канистры для хранения. Изучение РСУ и БТТ
  • tweak: изменение цен канистр

@github-actions github-actions bot added No C# Changes: Localization Изменение локализации labels Aug 21, 2024
Bolper
Bolper requested changes Aug 21, 2024
View reviewed changes
Copy link
Contributor

@Bolper Bolper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Поверхностное ревью.
Проведу полноценное попозже

Resources/Locale/en-US/research/technologies.ftl Outdated Show resolved Hide resolved
Resources/Locale/ru-RU/research/technologies.ftl Outdated Show resolved Hide resolved
@Flopochka Flopochka requested a review from Bolper August 21, 2024 18:57
@Bolper
Copy link
Contributor

Bolper commented Aug 21, 2024

Screenshot_2024-08-21-22-36-17-027_com android chrome
Чиним

Bolper
Bolper requested changes Aug 21, 2024
View reviewed changes
Copy link
Contributor

@Bolper Bolper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Так... по идеи все... Я конечно сейчас никакой и ревью на сегодня от меня больше на этом ПРе не ждите. Ночь

Resources/Locale/ru-RU/ADT/research/technologies.ftl Outdated Show resolved Hide resolved
Resources/Prototypes/Research/industrial.yml Outdated Show resolved Hide resolved
Resources/Prototypes/ADT/Recipes/Lathes/misc.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Entities/Structures/Machines/lathe.yml Outdated Show resolved Hide resolved
@Bolper
Copy link
Contributor

Bolper commented Aug 21, 2024

Screenshot_2024-08-21-22-36-17-027_com android chrome Чиним

Хотя... Наверно не в твоем ПРе беда, перезапущу тест

@Bolper Bolper added Status: Awaiting Changes Ожидание изменений and removed Status: Needs Review labels Aug 21, 2024
@Lonkuk
Copy link
Contributor

Lonkuk commented Aug 21, 2024 •
edited
Loading

image
Жалуется что есть изучения, но невозможно сделать на автолатах.
Всё что изучается нужно добавить в Resources\Prototypes\Entities\Structures\Machines\lathe.yml

@Bolper
Copy link
Contributor

Bolper commented Aug 22, 2024

image
Жалуется что есть изучения, но невозможно сделать на автолатах.
Всё что изучается нужно добавить в Resources\Prototypes\Entities\Structures\Machines\lathe.yml

А да, красавчик
Так и есть.

@Flopochka Flopochka requested review from 1Stepka1 and Bolper August 22, 2024 17:53
@github-actions github-actions bot added Status: Needs Review and removed Status: Awaiting Changes Ожидание изменений labels Aug 22, 2024
Bolper
Bolper requested changes Aug 23, 2024
View reviewed changes
Copy link
Contributor

@Bolper Bolper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я душнила я гандон

Resources/Prototypes/Entities/Structures/Storage/Canisters/gas_canisters.yml Show resolved Hide resolved
@Flopochka Flopochka requested a review from Bolper August 23, 2024 10:32
Bolper
Bolper approved these changes Aug 23, 2024
View reviewed changes
Copy link
Contributor

@Bolper Bolper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Как хорошо быть гандоном

@Bolper
Copy link
Contributor

Bolper commented Aug 23, 2024

@Flopochka У тебя тесты сдохли

@PyotrIgn PyotrIgn added this pull request to the merge queue Aug 23, 2024
Merged via the queue into AdventureTimeSS14:master with commit bf99abf Aug 23, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bolper Bolper Bolper approved these changes

@FaDeOkno FaDeOkno FaDeOkno left review comments

@1Stepka1 1Stepka1 Awaiting requested review from 1Stepka1

Assignees
No one assigned
Labels
Changes: Localization Изменение локализации No C#
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Flopochka @Bolper @Lonkuk @1Stepka1 @FaDeOkno @PyotrIgn