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

[VPRPVP] Emergency features / Snake Scales Fix. MCHPVP quickfix. #327

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

edewen
Copy link
Contributor

@edewen edewen commented Feb 2, 2025

Added The buff from snake scales to the list of when to ignore the emergency features. Basically, Snake scales is active and none of those buttons can be pressed until you backlash to end it, or it ends on its own. The features were preventing you from backlashing to end it.

Added Slither to the ignored movement action ids as well because I'm good like that.

Jr pvp dev when?

Added target check to mch pvp burst as a quick fix Tauren mentioned. It encompasses all of the burst mode instead of the couple lines he already added so it wont use analysis without a target and potentially waste the buffed tool attack.

Removed blazing shot option while I was at it, the game inherently changes the burst shot to blazing shot. You don't have a choice. Even with the option off, it does it because of the game itself. The option served no purpose.

@edewen edewen requested a review from Nik-Potokar as a code owner February 2, 2025 01:45
Copy link

semanticdiff-com bot commented Feb 2, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  WrathCombo/Combos/PvP/MCHPVP.cs  2% smaller
  WrathCombo/Combos/PvP/PVPCommon.cs  1% smaller
  WrathCombo/Combos/CustomComboPreset.cs  0% smaller

@zbee zbee linked an issue Feb 2, 2025 that may be closed by this pull request
@edewen edewen changed the title [VPRPVP] Emergency features / Snake Scales Fix [VPRPVP] Emergency features / Snake Scales Fix. MCHPVP quickfix. Feb 2, 2025
@Akechi-kun Akechi-kun added the Fixes This will fix a current issue label Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixes This will fix a current issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Viper Pvp Suggestion for Backlash
2 participants