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

BUFFS maille armor values #718

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

kokolo21
Copy link

@kokolo21 kokolo21 commented Feb 10, 2025

About The Pull Request

All this does is make Maille armor no longer the worst armor in the game. This was likely not by design but by oversight. It was weaker than the literal define for "weak" armor. It did not follow any logic.

Why It's Good For The Game

Makes maille armor follow the logical progression shown in other armor categories. It now is no longer the worst armor in the game.

Pre-Merge Checklist

  • You tested this on a local server.
  • This code did not runtime during testing.
  • You documented all of your changes.

fixes maille armor. It now no longer the worst armor in the game.
@tontyGH tontyGH added the balance surely this is good for the game label Feb 12, 2025
@tontyGH tontyGH changed the title (FIX) Fixes maille armor values BUFFS maille armor values Feb 12, 2025
@Sylphily
Copy link

not sure why this was changed to "buff" since this is a literal fix to the hell that is maille

@tontyGH
Copy link
Collaborator

tontyGH commented Feb 12, 2025

not sure why this was changed to "buff" since this is a literal fix to the hell that is maille

It is not broken... it is underpowered
This is a codebase, we use "fix" as in "make thing work again"

@tontyGH tontyGH added the active test merge This PR has been TMed to the main server label Feb 12, 2025
@tontyGH tontyGH merged commit b92d983 into Vanderlin-Tales-Of-Wine:main Feb 13, 2025
@tontyGH tontyGH removed the active test merge This PR has been TMed to the main server label Feb 13, 2025
Golubian pushed a commit to Golubian/Vanderlin that referenced this pull request Feb 14, 2025
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- Note: PRs including balance changes authored by anyone other than
maintainers and official devs will not be considered. -->

## About The Pull Request

<!-- Describe The Pull Request. Please be sure every change is
documented or this can delay review or prevent the PR from being merged!
-->
All this does is make Maille armor no longer the worst armor in the
game. This was likely not by design but by oversight. It was weaker than
the literal define for "weak" armor. It did not follow any logic.

## Why It's Good For The Game

<!-- Please add a short description of why you think these changes would
benefit the game. If you can't justify it in words, it might not be
worth adding. -->
Makes maille armor follow the logical progression shown in other armor
categories. It now is no longer the worst armor in the game.

## Pre-Merge Checklist
<!-- Don't bother filling these in while creating your Pull Request,
just click the checkboxes after the Pull Request is opened and you are
redirected to the page. -->
- [x] You tested this on a local server.
- [x] This code did not runtime during testing.
- [x] You documented all of your changes.
<!-- Neither the compiler nor workflow checks are perfect at detecting
runtimes and errors. It is important to test your code/feature/fix
locally. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
balance surely this is good for the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants