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

Swift fix #13

Merged
merged 2 commits into from
Jun 7, 2024
Merged

Conversation

800maximum123
Copy link
Collaborator

ПР актуализирует данные по Резоми до Лора ФД.

close #0

Чейнджлог

🆑SWIFT
tweak: Подправил данные по Резоми, согласно актуальному лору
/🆑
  • Pull Request полностью завершен, мне не нужна помощь чтобы его закончить.
  • Я внимательно прочитал все свои изменения и багов в них не нашел.
  • Я запускал сервер со своими изменениями локально и все протестировал.

800maximum123 and others added 2 commits June 7, 2024 18:20
Co-Authored-By: SWIFTWAVE0 <[email protected]>
Copy link

@VoiceInYourHead VoiceInYourHead left a comment

Choose a reason for hiding this comment

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

Исправить. Затем замёржу.

<!-- Название мода. Не важно на русском или на английском. -->
## Резоми

ID мода: RESOMI

Choose a reason for hiding this comment

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

Айди мода совпадает с таковым у оригинального. Не критично, ведь мы на вряд ли будем включать их одновременно(зачем?), но лучше дописать банальное RESOMI_FD.

Изменений нет - напиши "Отсутствуют"
-->

### Дефайны

Choose a reason for hiding this comment

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

Изменить список добавляемых дефайнов, убрав отсюда всё лишнее и добавив нужное.

-->

### Авторы:

Choose a reason for hiding this comment

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

Дописать себя в автора!

#define CULTURE_RESOMI_NEWGENERATION "New generation"
#define CULTURE_RESOMI_LOSTCOLONYRICH "A native of a thriving lost colony"
#define CULTURE_RESOMI_LOSTCOLONYPOOR "A native of a impoverished lost colony"
#define CULTURE_RESOMI_SKRELL "Skrellian Adoptive"

Choose a reason for hiding this comment

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

Давно скреллы усыновляют резоми? Это в целом касается файла - подогнать его под нашу инфу, удалить всё лишнее.

@@ -0,0 +1,19 @@
#define CULTURE_RESOMI_TEMALI "Temali"

Choose a reason for hiding this comment

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

Перенести всё отсюда в mods/resomi_fd/code/datum/_defines.dm, вообще не понял зачем ты создал отдельный файл и пихал это сюда, если ты всё равно создал наш независимый от того мод.

@VoiceInYourHead
Copy link

Ладно похуй сделаю сам

@VoiceInYourHead VoiceInYourHead merged commit 80c0bea into RepoStash:corecode-rewrite Jun 7, 2024
2 checks passed
@800maximum123
Copy link
Collaborator Author

Я не дотянулся еще ручками до этого. Но ладно.

VoiceInYourHead added a commit that referenced this pull request Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants