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

[Ability] Heatproof now reduces burn damage by half #3524

Merged
merged 5 commits into from
Aug 13, 2024

Conversation

DayKev
Copy link
Collaborator

@DayKev DayKev commented Aug 13, 2024

What are the changes?

Heatproof now reduces burn damage by half.

Why am I doing these changes?

cf https://bulbapedia.bulbagarden.net/wiki/Heatproof_(Ability) and https://www.smogon.com/forums/threads/pokemon-sun-moon-battle-mechanics-research.3586701/

What did change?

A new ReduceBurnDamageAbAttr was created and applied to the burn damage calculation in phases.ts.

How to test the changes?

Override your Pokémon to have Heatproof and be burned and pass a turn. Then do the same without Heatproof. Compare the damage.
npm run test heatproof

Checklist

  • I'm using beta as my base branch
  • There is no overlap with another PR?
  • The PR is self-contained and cannot be split into smaller PRs?
  • Have I provided a clear explanation of the changes?
  • Have I considered writing automated tests for the issue?
  • Have I tested the changes (manually)?
    • Are all unit tests still passing? (npm run test)
  • [ ] Are the changes visual?
    • [ ] Have I provided screenshots/videos of the changes?

@DayKev DayKev added the Ability Affects an ability label Aug 13, 2024
Copy link
Collaborator

@frutescens frutescens left a comment

Choose a reason for hiding this comment

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

Looks good.

@f-fsantos f-fsantos merged commit 39e7591 into pagefaultgames:beta Aug 13, 2024
4 checks passed
@DayKev DayKev deleted the finish-heatproof branch August 13, 2024 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ability Affects an ability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants