Vulphair #120
Merged
Vulphair #120
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This reverts commit a06c102.
|
Зачем у всех путей причесок есть vulp_ в начале, если там и так ясно, что они для вульп? |
Копипаст я так понял |
Furrior
requested changes
Aug 1, 2023
|
А еще тесты упали |
Furrior
reviewed
Aug 3, 2023
Потому что они для вульп XyliPizdish |
|
Так же, вижу дубликату уже существующих причесок, непонятно зачем. Не вижу, что бы спрайт отличался у них |
Furrior
approved these changes
Aug 11, 2023
Furrior
pushed a commit
that referenced
this pull request
Sep 20, 2023
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то может пойти не так. --> <!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. --> ## Что этот PR делает Пачка новых волос для расы Vulpkanin. Мужские и женские варианты, честно позаимствованы и адаптированы под расу. <!-- Вкратце опишите изменения, которые вносите. --> <!-- Опишите **все** изменения, так как противное может сказаться на рассмотрении этого PR'а! --> <!-- Если вы исправляете Issue, добавьте "Fixes #1234" (где 1234 - номер Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после принятия PR'а. --> ## Почему это хорошо для игры Причёски круто <!-- Опишите, почему, по вашему, следует добавить эти изменения в игру. --> ## Изображения изменений <!-- Если вы не меняли карту или спрайты, можете опустить эту секцию. Если хотите, можете вставить видео. -->  ## Тестирование <!-- Как вы тестировали свой PR, если делали это вовсе? --> Работает ## Changelog Gatto Протестировал 🆑 add: У расы вульпканин теперь больше спрайтов для волос /🆑 <!-- Оба 🆑'а должны быть на месте, что-бы чейнджлог работал! Вы можете написать свой ник справа от первого 🆑, если хотите. Иначе будет использован ваш ник на ГитХабе. --> <!-- Вы можете использовать несколько записей с одинаковым префиксом (Они используются только для иконки в игре) и удалить ненужные. Помните, что чейнджлог должен быть понятен обычным игроком. --> <!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы можете исключить всю секцию. -->
Furrior
pushed a commit
that referenced
this pull request
Sep 21, 2023
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то может пойти не так. --> <!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. --> ## Что этот PR делает Пачка новых волос для расы Vulpkanin. Мужские и женские варианты, честно позаимствованы и адаптированы под расу. <!-- Вкратце опишите изменения, которые вносите. --> <!-- Опишите **все** изменения, так как противное может сказаться на рассмотрении этого PR'а! --> <!-- Если вы исправляете Issue, добавьте "Fixes #1234" (где 1234 - номер Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после принятия PR'а. --> ## Почему это хорошо для игры Причёски круто <!-- Опишите, почему, по вашему, следует добавить эти изменения в игру. --> ## Изображения изменений <!-- Если вы не меняли карту или спрайты, можете опустить эту секцию. Если хотите, можете вставить видео. -->  ## Тестирование <!-- Как вы тестировали свой PR, если делали это вовсе? --> Работает ## Changelog Gatto Протестировал 🆑 add: У расы вульпканин теперь больше спрайтов для волос /🆑 <!-- Оба 🆑'а должны быть на месте, что-бы чейнджлог работал! Вы можете написать свой ник справа от первого 🆑, если хотите. Иначе будет использован ваш ник на ГитХабе. --> <!-- Вы можете использовать несколько записей с одинаковым префиксом (Они используются только для иконки в игре) и удалить ненужные. Помните, что чейнджлог должен быть понятен обычным игроком. --> <!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы можете исключить всю секцию. -->
Furrior
pushed a commit
that referenced
this pull request
Oct 22, 2023
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то может пойти не так. --> <!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. --> ## Что этот PR делает Пачка новых волос для расы Vulpkanin. Мужские и женские варианты, честно позаимствованы и адаптированы под расу. <!-- Вкратце опишите изменения, которые вносите. --> <!-- Опишите **все** изменения, так как противное может сказаться на рассмотрении этого PR'а! --> <!-- Если вы исправляете Issue, добавьте "Fixes #1234" (где 1234 - номер Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после принятия PR'а. --> ## Почему это хорошо для игры Причёски круто <!-- Опишите, почему, по вашему, следует добавить эти изменения в игру. --> ## Изображения изменений <!-- Если вы не меняли карту или спрайты, можете опустить эту секцию. Если хотите, можете вставить видео. -->  ## Тестирование <!-- Как вы тестировали свой PR, если делали это вовсе? --> Работает ## Changelog Gatto Протестировал 🆑 add: У расы вульпканин теперь больше спрайтов для волос /🆑 <!-- Оба 🆑'а должны быть на месте, что-бы чейнджлог работал! Вы можете написать свой ник справа от первого 🆑, если хотите. Иначе будет использован ваш ник на ГитХабе. --> <!-- Вы можете использовать несколько записей с одинаковым префиксом (Они используются только для иконки в игре) и удалить ненужные. Помните, что чейнджлог должен быть понятен обычным игроком. --> <!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы можете исключить всю секцию. -->
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Что этот PR делает
Пачка новых волос для расы Vulpkanin.
Мужские и женские варианты, честно позаимствованы и адаптированы под расу.
Почему это хорошо для игры
Причёски круто
Изображения изменений
Тестирование
Работает
Changelog
Gatto
Протестировал
🆑
add: У расы вульпканин теперь больше спрайтов для волос
/🆑