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
Removes Space Ninjas #492
Removes Space Ninjas #492
Conversation
Hitler. |
Hit- actually this is probably for the best |
"Showing 43 changed files with 68 additions and 3,930 deletions." That's 3,930 less lines of some of the snowflakiest code we have. |
Yes, I am a fun-ruining SuperHitler. Until I re-code them. |
We should recode Space Ninjas, but better than they were before. |
That's the goal. |
That's what I said I'd do, both in the comments and in the PR body. |
Still need to kill off every snowflake ability his energy sword has, as well, before this gets merged. ie: obj/item/weapon/melee/energy/blade |
Ahahaahah well holy hell its finally gone! |
Unfortunately, Fox, the energy blade has to stay. It's the only Ninja ability I haven't thought of a way of a way to modularize, due to the fact it needs to both bludgeon people and slice open doors. (Causing the NOBLUDGEON flag coupled with an afterattack proc to be unviable as an option) If anyone can think of a way to un-snowflake the e-blade code, I'd be more than happy to remove it, but until then I'm not gonna remove it because it's gonna end up being implemented in the exact same way it currently is. |
What about splitting it up into two items? The energy blade which can be used to slice people and walls, and an energy card which can be used on doors only. Activating the blade turns it into a card, and vice versa. Kind of like guns and their modes. |
@SamCroswell Well, one way around that could be to give him a reskinned emag, as that already accomplishes (more or less) the same exact thing, with few exceptions (cameras). Either case, if you feel it has to stay, fair enough! |
rip in piss |
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то может пойти не так. --> <!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. --> ## Что этот PR делает <!-- Вкратце опишите изменения, которые вносите. --> <!-- Опишите **все** изменения, так как противное может сказаться на рассмотрении этого PR'а! --> <!-- Если вы исправляете Issue, добавьте "Fixes ParadiseSS13#1234" (где 1234 - номер Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после принятия PR'а. --> Увеличивает world.view с 15х15 до 19х19. Вроде бы не ломает параллакс, сей работает по горизонтали как нужно, по вертикали вряд ли исправить (слышишь дальше, чем видишь, на 2 турфа) ## Почему это хорошо для игры <!-- Опишите, почему, по вашему, следует добавить эти изменения в игру. --> Если видишь, как кто-то говорит, стоит услышать их ## Тестирование <!-- Как вы тестировали свой PR, если делали это вовсе? --> сей-сей, полетел по космосу, вроде работает ## Changelog :cl: tweak: Увеличение world.view с 15x15 до 19x19, что позволит слышать на вайдскрине людей на границе экрана. /:cl: <!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы можете написать свой ник справа от первого :cl:, если хотите. Иначе будет использован ваш ник на ГитХабе. --> <!-- Вы можете использовать несколько записей с одинаковым префиксом (Они используются только для иконки в игре) и удалить ненужные. Помните, что чейнджлог должен быть понятен обычным игроком. --> <!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы можете исключить всю секцию. -->
Removes Space Ninjas entirely, pending re-implementation in a more maintainable and less god-awful manner.
Ordinarily I'd discuss this before dreaming of posting the PR, but since Ninja is borderline adminspawn-only anyhow, I don't think it'll be an issue.
Feel free to call me literally Hitler in the comments if you disagree. (Or reasonably voice your opinion. That works too.)