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

Armour Rework (lyndo please cook) #1687

Closed
wants to merge 15 commits into from

Conversation

TadJohnson00
Copy link
Contributor

@TadJohnson00 TadJohnson00 commented Aug 26, 2024

About the PR

Adjusted armour values (touches delta, wiz, and nyano files) in anticipation of #1672

Why / Balance

See above

Technical details

n/a

Media

n/a

Requirements

  • I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

n/a
Changelog

🆑 TJohnson & Lyndomen

  • tweak: up to lyndo to decide here (thx luv u)

@github-actions github-actions bot added the Changes: YML Changes any yml files label Aug 26, 2024
@Lyndomen
Copy link
Contributor

Lyndomen commented Aug 29, 2024

This is intended to go live with one or two other PRs, shock damage + blunt damage stamina rebuild + weapon rebalance

#1672

stun meta btfo

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Seems good, but you should rewrite some of the comments so the change is documented from like, 0.75 to 0.85 just as a reference of how it was before. Having everything just say "General armor rebalance" gives no future yaml people who want to change this any clue how it was and kinda forces them to look up the PR.

@TadJohnson00
Copy link
Contributor Author

Added comments as requested

@ghost
Copy link

ghost commented Sep 4, 2024

Keep cooking 👍

@github-actions github-actions bot added the S: Merge Conflict Fix your PR! label Sep 6, 2024
Copy link
Contributor

github-actions bot commented Sep 6, 2024

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

@TadJohnson00 TadJohnson00 deleted the armourrework branch September 28, 2024 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files S: Merge Conflict Fix your PR!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants