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

Revert grass tiles texture #2272

Closed
wants to merge 1 commit into from
Closed

Revert grass tiles texture #2272

wants to merge 1 commit into from

Conversation

Ko4ergaPunk
Copy link
Collaborator

Описание PR

Ублюдство #28868

Медиа

Проверки

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

Изменения

Не весело

@Ko4ergaPunk Ko4ergaPunk requested a review from Morb0 as a code owner June 16, 2024 15:13
@github-actions github-actions bot added Status: Needs Review This PR requires new reviews before it can be merged 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 and removed Status: Needs Review This PR requires new reviews before it can be merged labels Jun 16, 2024
@github-actions GitHub Actions
Copy link
Contributor

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

Resources/Textures/Objects/Tiles/tile.rsi

State Old New Status
astrograss Modified
grass Modified
grassjungle Modified

Resources/Textures/Tiles/Planet/dirt.rsi

State Old New Status
dirt Modified

@lzk228
Copy link
Collaborator

lzk228 commented Jun 16, 2024

а space-wizards/space-station-14#26449 не волнует?

@Ko4ergaPunk
Copy link
Collaborator Author

а space-wizards/space-station-14#26449 не волнует?

Нет

@NotSoDana
Copy link
Contributor

а space-wizards/space-station-14#26449 не волнует?

Нет, не волнует

@Hemomancer
Copy link

Нет

@lzk228
Copy link
Collaborator

lzk228 commented Jun 16, 2024

image
тогда почему мы можем?

@Sergan31
Copy link

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

@Pok27
Copy link
Contributor

Pok27 commented Jun 16, 2024

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

@Ko4ergaPunk
Copy link
Collaborator Author

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

image
image
Да тут вкуснотища

@Ko4ergaPunk
Copy link
Collaborator Author

image тогда почему мы можем?

Ты задал очень глупый вопрос. Ответь почему не можем?
image

@lapatison
Copy link
Contributor

image тогда почему мы можем?

Ты задал очень глупый вопрос. Ответь почему не можем? image

Очень глупый ответ. Раз это пизженные текстуры из другой игры, то в лицензии ложь.

Ну или можешь пруфануть, что эти текстуры действительно под этой лицензией.

@Sergan31
Copy link

Я успел зайти в игру и понял весь прикол. Остаётся надеяться на то, что новые текстурки будут не хуже

@Morb0
Copy link
Member

Morb0 commented Jun 17, 2024

image тогда почему мы можем?

Ты задал очень глупый вопрос. Ответь почему не можем? image

Очень глупый ответ. Раз это пизженные текстуры из другой игры, то в лицензии ложь.

Ну или можешь пруфануть, что эти текстуры действительно под этой лицензией.

Это так, лицензию просто воткнули. Нужен нормальный респрайт или найти с открытой лицензией.

@Morb0 Morb0 closed this Jun 17, 2024
@Ko4ergaPunk Ko4ergaPunk deleted the Revert-grass-tiles branch June 30, 2024 10:50
@Ko4ergaPunk Ko4ergaPunk mentioned this pull request Jul 5, 2024
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Morb0 Morb0 Awaiting requested review from Morb0 Morb0 is a code owner

Assignees
No one assigned
Labels
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
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@Ko4ergaPunk @lzk228 @NotSoDana @Hemomancer @Sergan31 @Pok27 @lapatison @Morb0