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

Removes useless t-ray scanners #1784

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

larentoun
Copy link
Collaborator

@larentoun larentoun commented Feb 6, 2025

Что этот PR делает

Почему это хорошо для игры

Удалил t-ray сканеры, которые использовались только для фурукайного ниндзи, который нам не нужен.

Changelog

🆑
del: Удаляет лишние t-ray сканеры. Они ничего не делают, описание путает.
/:cl:

Summary by Sourcery

Chores:

  • Remove t-ray scanners that were only used for the furukai ninja, which is no longer needed.

Copy link

sourcery-ai bot commented Feb 6, 2025

Reviewer's Guide by Sourcery

This PR removes the t-ray scanners, which were only used for the removed furukai ninja.

No diagrams generated as the changes look simple and do not need a visual representation.

File-Level Changes

Change Details Files
Removal of t-ray scanner files.
  • Deleted the scanners.dm file.
  • Deleted the equipment_designs.dm file.
modular_ss220/devices/code/items/scanners.dm
modular_ss220/devices/code/research_designs/equipment_designs.dm
Removal of t-ray scanner entries.
  • Removed t-ray scanner entries from the _devices.dme file.
modular_ss220/devices/_devices.dme

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!
  • Generate a plan of action for an issue: Comment @sourcery-ai plan on
    an issue to generate a plan of action for it.

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@ss220app ss220app bot added the 📜 CL валиден Этот чейнджлог будет успешно опубликован label Feb 6, 2025
@HellPon3
Copy link

HellPon3 commented Feb 6, 2025

Вообще то мы эти Тиреи использовали для поднятия технологий в рнд

@AyIong AyIong merged commit 63ddf5c into ss220club:master Feb 6, 2025
22 checks passed
MKardinal added a commit to MKardinal/Paradise-SS220 that referenced this pull request Feb 8, 2025
MKardinal added a commit to MKardinal/Paradise-SS220 that referenced this pull request Feb 8, 2025
@MKardinal MKardinal mentioned this pull request Feb 8, 2025
m-dzianishchyts pushed a commit that referenced this pull request Feb 9, 2025
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то
может пойти не так. -->
<!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. -->

## Что этот PR делает
Отменяет действие [PR #1784
](#1784). Обновляет
описания T-ray сканеров. Изменяет стоимость исследования и производства
наиболее продвинутых вариантов. Удаляет неиспользуемый в игре хайриск
вариант. Балансирует уровни технологий, необходимые для создания
сканеров, и получаемые от их разбора.

<!-- Вкратце опишите изменения, которые вносите. -->
<!-- Опишите **все** изменения, так как противное может сказаться на
рассмотрении этого PR'а! -->
<!-- Если вы исправляете Issue, добавьте "Fixes #xxxx" (где xxxx - номер
Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после
принятия PR'а. -->

## Почему это хорошо для игры
T-ray сканеры - полезный инструмент для инженеров, у них интересная
ветка развития с постепенным улучшением характеристик. Наиболее
продвинутые варианты открываются при высоком уровне технологий, а их
производство требует ценных материалов. Убирается путь спидрана
технологий при помощи сканеров.

<!-- Опишите, почему, по вашему, следует добавить эти изменения в игру.
-->

## Изображения изменений
<!-- Если вы не меняли карту или спрайты, можете опустить эту секцию.
Если хотите, можете вставить видео. -->

## Тестирование
Запустил локальный сервер, заспавнил сканеры, посмотрел описания и
результат разбора в научном анализаторе.
<!-- Как вы тестировали свой PR, если делали это вовсе? -->

## Changelog

:cl:
add: T-ray сканеры добавлены обратно.
tweak: Сбалансированы уровни технологий, необходимые для создания
сканеров, и получаемые при помощи их разбора. Теперь поднять
максимальный уровень magnets при помощи последовательного разбора
сканеров не получится.
del: Неиспользуемый хайриск вариант сканера удален.
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->

## Summary by Sourcery

Reintroduce T-ray scanners with updated descriptions and adjusted
research and production costs for advanced versions. Remove the unused
high-risk scanner variant.

New Features:
- Reintroduced T-ray scanners.

Tests:
- Tested T-ray scanners on a local server to verify descriptions and
functionality.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📜 CL валиден Этот чейнджлог будет успешно опубликован
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants