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

addBackShield #2233

Merged
merged 2 commits into from
Nov 23, 2024
Merged

addBackShield #2233

merged 2 commits into from
Nov 23, 2024

Conversation

cherborr
Copy link

@cherborr cherborr commented Nov 14, 2024

Описание PR

Добавление функционала экипировки щита на спину с последующим понижением коэффициента защиты.
Сделано по задаче

Медиа

Проверки

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

Изменения

@github-actions github-actions bot added the Changes: C# Изменение затронуло файлы ".cs" label Nov 14, 2024
@Kirus59 Kirus59 requested a review from stalengd November 14, 2024 22:31
Copy link
Collaborator

@stalengd stalengd left a comment

Choose a reason for hiding this comment

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

Небольшой косяк с кодировкой, а так вроде норм

@@ -1,4 +1,4 @@
using System.Linq;
using System.Linq;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Нехорошо получилось, это UTF-8 перешёл в UTF-8 with BOM или наоборот

@stalengd stalengd self-assigned this Nov 14, 2024
@cherborr cherborr requested a review from stalengd November 22, 2024 18:17
@stalengd stalengd merged commit 0f0ea4c into SerbiaStrong-220:master Nov 23, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants