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

CHAINSAW!!!!! #135

Closed
wants to merge 4 commits into from
Closed

CHAINSAW!!!!! #135

wants to merge 4 commits into from

Conversation

Ratyyy
Copy link
Collaborator

@Ratyyy Ratyyy commented Jul 25, 2024 •
edited
Loading

Описание PR

полный рефактор бензопилы, теперь она имеет крафт, постоянный звук при включении, потребление топлива, увеличение урона

сделано в качестве части апдейта самодельного вооружения
большая часть кода в этом пр-е написана milenwolf
Медиа

Проверки

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

Изменения

@github-actions github-actions bot added Changes: Sprites Изменение спрайтов Changes: Localization Изменение локализации labels Jul 25, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jul 25, 2024 •
edited
Loading

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

Resources/Textures/Objects/Weapons/Melee/chainsaw.rsi

State Old New Status
icon Modified
inhand-left Modified
inhand-right Modified
wielded-inhand-left Modified
wielded-inhand-right Modified
chainsaw_on Added
inhand-left-on Added
inhand-right-on Added

Edit: diff updated after ef89db8

@Ratyyy Ratyyy enabled auto-merge July 25, 2024 16:59
@1Stepka1
Copy link
Collaborator

1Stepka1 commented Jul 26, 2024 •
edited
Loading

Я без понятия для чего нужен этот апдейт и что он делает и почему?
А если к сути вопроса то лучше изменить описание реквеста чтобы было ну более понятно что и зачем это делается.
На данный момент без этого я не могу провести ревью

@Ratyyy
Copy link
Collaborator Author

Ratyyy commented Jul 26, 2024

Я без понятия для чего нужен этот апдейт и что он делает и почему? А если к сути вопроса то лучше изменить описание реквеста чтобы было ну более понятно что и зачем это делается. На данный момент без этого я не могу провести ревью

готово, описание есть

1Stepka1
1Stepka1 requested changes Jul 26, 2024
View reviewed changes
Copy link
Collaborator

@1Stepka1 1Stepka1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Что сказать у визардов и корваксов такого я не вижу Resources/Prototypes/Recipes/Crafting/Graphs/improvised/chainsaw.yml и - type: construction name: Chainsaw id: chainsaw в Resources/Prototypes/Recipes/Crafting/improvised.yml то это перемещается в нашу папочку ADT.
Вижу так же много тегов в Resources/Prototypes/tags.yml все что не наши переезжают в Prototypes/ADT/tags.yml .
В Resources/Locale/ru-RU/ss14-ru/prototypes/entities/objects/weapons/melee/chainsaw.ftl укажи где начинаются наши изменения к примеру #ADT chainsaw

@github-actions github-actions bot added Status: Awaiting Changes Ожидание изменений Merge Conflict labels Jul 26, 2024
@github-actions GitHub Actions
Copy link
Contributor

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

@Ratyyy
Merge branch 'master' of https://github.com/AdventureTimeSS14/space_s…
ef89db8 
…tation_ADT into chainsaw

# Conflicts:
#	Content.Shared/Item/ItemToggle/ItemToggleSystem.cs
@Bolper
Copy link
Contributor

Bolper commented Aug 21, 2024

@Ratyyy Работы по ПРу ведутся?

@Bolper Bolper added the In progress В процессе выполнения label Aug 30, 2024
@Ratyyy
Copy link
Collaborator Author

Ratyyy commented Aug 31, 2024

@Ratyyy Работы по ПРу ведутся?

по-тихоньку

@Schrodinger71 Schrodinger71 marked this pull request as draft September 6, 2024 22:12
auto-merge was automatically disabled September 6, 2024 22:12

Pull request was converted to draft

@Darkiich
Copy link
Contributor

Закрываю PR по причине долгого отсутствия актива

@Darkiich Darkiich closed this Oct 26, 2024
@Schrodinger71 Schrodinger71 deleted the chainsaw branch October 27, 2024 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@1Stepka1 1Stepka1 1Stepka1 requested changes

Assignees

@Ratyyy Ratyyy

Labels
Changes: Localization Изменение локализации Changes: Sprites Изменение спрайтов In progress В процессе выполнения Status: Awaiting Changes Ожидание изменений
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Ratyyy @1Stepka1 @Bolper @Darkiich