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.

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

finishing command turtlenecks #2290

Closed
wants to merge 28 commits into from
Closed

finishing command turtlenecks #2290

wants to merge 28 commits into from

Conversation

Deenkaide
Copy link
Contributor

@Deenkaide Deenkaide commented Jun 18, 2024

Описание PR

Допиливание водолазок

Медиа
гифки

Проверки

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

Изменения
🆑 Deenkaide

  • add: Водолазки в принтер одежды
  • add: Перевод водолазкам
  • tweak: Костюм НРа переименован в классический

@github-actions github-actions bot added Changes: No C# Code has not been changed Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation labels Jun 18, 2024
Copy link
Contributor

github-actions bot commented Jun 18, 2024

RSI Diff Bot; head commit e071c82 merging into ae6ed32
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Structures/Power/substation.rsi

State Old New Status
charging Modified
dead Modified
full Modified
screen Modified
screen_wall Modified
substation Modified
substation_static Modified
substation_wall Modified
substation_wall_static Modified

Resources/Textures/Structures/Wallmounts/posters.rsi

State Old New Status
poster9_legit Modified

Edit: diff updated after e071c82

@Deenkaide
Copy link
Contributor Author

Я понял как создать Драфт ПР, а ещё конфликт я решу позже, после того, как вещи добавят в лодауты

Copy link
Contributor

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

@github-actions github-actions bot added the Merge Conflict PR have unsolved merge conflicts label Jun 18, 2024
@MureixloI
Copy link
Contributor

MureixloI commented Jun 18, 2024

Водолазка СИ не подходит под палитру его, а также водолазка НРа получилось как-то мыльной, типо 3 оттенок слишком яркий и сливается со вторым по сравнению с 4-м.

@lzk228
Copy link
Collaborator

lzk228 commented Jun 18, 2024

перевод можно оставить на лапатисона

@Deenkaide
Copy link
Contributor Author

Водолазка СИ не подходит под палитру его, а также водолазка НРа получилось как-то мыльной, типо 3 оттенок слишком яркий и сливается со вторым по сравнению с 4-м.

Водолазка СИ уже была такой, посмотрю, что могу сделать.
По поводу НРа... опять таки попробую изменить

@Deenkaide
Copy link
Contributor Author

перевод можно оставить на лапатисона

Ну, он уже залит, если что исправит/изменит

@Deenkaide
Copy link
Contributor Author

Не знаю на сколько сильно улучшилась ситуация у водолазки НРа, но СИ перекрашен в цвета его обычного комбинезона

@Deenkaide
Copy link
Contributor Author

Что-то меня начинает напрягать всё растущий список конфликтов =)

@Deserty0
Copy link
Contributor

Что-то меня начинает напрягать всё растущий список конфликтов =)

потому что их чинить вовремя надо...

@Deenkaide
Copy link
Contributor Author

Deenkaide commented Jun 19, 2024

потому что их чинить вовремя надо...

Слишком легко)
У меня есть предположение, что это из-за того, что я не сменил ветку перед открытием этого ПРа после конфуза с прошлым

@Deenkaide
Copy link
Contributor Author

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

@themanyfaceddemon
Copy link

themanyfaceddemon commented Jun 20, 2024

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

git revert коммит вам в помощь

@themanyfaceddemon
Copy link

Так. Стоп. Куда я вообще ответил и на какой вопрос...

@themanyfaceddemon
Copy link

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

Так. Я чуть проснулся.
Если есть гитхаб десктоп - будет проще.
Через консоль прописываете на своей ветке git pull https://github.com/space-syndicate/space-station-14.git master
Смотрите в десктоп и решаете конфликты. Всё. Профит.

Иначе смотрим это

@lzk228
Copy link
Collaborator

lzk228 commented Jun 20, 2024

так автор челик открыл пр с той же ветки что и предыдущий даже не обновив её

@Deenkaide
Copy link
Contributor Author

Deenkaide commented Jun 20, 2024

так автор челик открыл пр с той же ветки что и предыдущий даже не обновив её

я понял, что сделал, слишком поздно. Сейчас у себя поэкспериментировал и у меня есть идея как это всё исправить

@github-actions github-actions bot added the Changes: Map Can be reviewed or fixed by people who are knowledgeable with mapping label Jun 20, 2024
…kaide/space-station-14 into command-turtlenecks"

This reverts commit a3a9cb2, reversing
changes made to d2eb8a0.
@Deenkaide
Copy link
Contributor Author

по моему хрень сделал... оно пошло не по плану

@Deenkaide
Copy link
Contributor Author

@lzk228, может я уже закрою этот ПР и залью нормально?

@lzk228
Copy link
Collaborator

lzk228 commented Jun 20, 2024

на всё воля твоя

@Deenkaide
Copy link
Contributor Author

Я окончательно доломал этот ПР, исправляю ошибки и создаю нормальный.

@Deenkaide Deenkaide closed this Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation Changes: Map Can be reviewed or fixed by people who are knowledgeable with mapping Changes: No C# Code has not been changed Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design Merge Conflict PR have unsolved merge conflicts
Projects
None yet
Development

Successfully merging this pull request may close these issues.