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

Correct Alliance member raising logic #618

Merged
merged 1 commit into from
Feb 24, 2025
Merged

Correct Alliance member raising logic #618

merged 1 commit into from
Feb 24, 2025

Conversation

LTS-FFXIV
Copy link
Member

This pull request includes changes to the RotationSolver/Updaters/TargetUpdater.cs file to improve the handling of hostile targets and death targets in the GetAllHostileTargets method. The most important changes include initializing deathAllianceHealers and deathAllianceSupports with deathParty and adding a new line for code readability.

Improvements to hostile targets handling:

Code readability improvements:

  • RotationSolver/Updaters/TargetUpdater.cs: Added a new line for better separation of methods.The initialization of deathAllianceHealers and deathAllianceSupports now includes the contents of deathParty, ensuring these lists start with the same elements as deathParty.

Added a new private static method GetPriorityDeathTarget to determine the highest priority target for raising from a list of valid targets, potentially improving the logic for handling death targets in the game.

The initialization of `deathAllianceHealers` and `deathAllianceSupports` now includes the contents of `deathParty`, ensuring these lists start with the same elements as `deathParty`.

Added a new private static method `GetPriorityDeathTarget` to determine the highest priority target for raising from a list of valid targets, potentially improving the logic for handling death targets in the game.
@LTS-FFXIV LTS-FFXIV merged commit 671a962 into main Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant