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

Вырезка инвертора квантового спина #776

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Вырезка инвертора квантового спина #776

wants to merge 6 commits into from

Conversation

Esco-dev
Copy link
Contributor

@Esco-dev Esco-dev commented Dec 4, 2024 •
edited
Loading

Кратное описание

Инвертор Квантового спина будет недоступен для производства, спавна и изучения

По какой причине

Бесполезная шмотка которой пользуются дурики чтобы позлить сб. Часто ломает отображения спрайтов и сама по себе вещь багованная.

Медиа(Видео/Скриншоты)

Проверки

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

Changelog

🆑 JunterFox

  • remove: Инвертор квантового спина удалён из игры.

@Esco-dev Esco-dev requested a review from VigersRay as a code owner December 4, 2024 07:02
@KaiserMaus
Copy link
Contributor

Сломан все такое но может стоит оставить в качестве админ вещи?

@mithail
Copy link

mithail commented Dec 4, 2024

Оставьте в админ вещах, может пригодится

@Esco-dev
Copy link
Contributor Author

Esco-dev commented Dec 4, 2024

Вряд ли кто то будет использовать инвертор когда есть:
А: tpto
Б: невидимый телепорт
Тем не менее, мой PR открыт мейнтейнерам, поменяют если потребуется. Все администраторы с которыми я общался сообщили что это ненужный в процессе ивентов предмет.

@github-actions GitHub Actions
Copy link
Contributor

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

@KaiserMaus
Copy link
Contributor

Сделай как сделали со скороходами
удалили Технологию и Крафты (под удалили имеется ввиду поставили #)

@Esco-dev
Copy link
Contributor Author

Esco-dev commented Jan 5, 2025

Сделай как сделали со скороходами удалили Технологию и Крафты (под удалили имеется ввиду поставили #)

Я так и сделал

@KaiserMaus
Copy link
Contributor

Сделай как сделали со скороходами удалили Технологию и Крафты (под удалили имеется ввиду поставили #)

Я так и сделал

Линтер чини

@Esco-dev
Copy link
Contributor Author

Esco-dev commented Jan 5, 2025

Пусть чинит тот кто уже 1к+ комитов вперед пропустил, на момент залива все рабочее было

@Esco-dev
Copy link
Contributor Author

Esco-dev commented Jan 5, 2025

Да и линтер там на оружие жалуется

@KaiserMaus
Copy link
Contributor

Да и линтер там на оружие жалуется

Так мержи просто сборку обнови у тебя отстала она

@Esco-dev Esco-dev marked this pull request as draft January 7, 2025 03:29
@Esco-dev
Merge branch 'master' into Swapper-bye
eed05ef 
# Conflicts:
#	Resources/Prototypes/Entities/Objects/Devices/swapper.yml
@github-actions github-actions bot added size/S and removed size/M labels Jan 8, 2025
@Esco-dev Esco-dev marked this pull request as ready for review January 8, 2025 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

Assignees
No one assigned
Labels
Нет C# size/S Статус: Несортированный
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Esco-dev @KaiserMaus @mithail