Skip to content

Small fixes to naxxramas strategy#2201

Merged
kadeshar merged 3 commits intomod-playerbots:test-stagingfrom
kadeshar:naxxramas-fixes
Mar 13, 2026
Merged

Small fixes to naxxramas strategy#2201
kadeshar merged 3 commits intomod-playerbots:test-stagingfrom
kadeshar:naxxramas-fixes

Conversation

@kadeshar
Copy link
Collaborator

Pull Request

Small fixes to naxxramas strategy


Complexity & Impact

Does this change add new decision branches?

    • No
    • Yes (explain below)

Does this change increase per-bot or per-tick processing?

    • No
    • Yes (describe and justify impact)

Could this logic scale poorly under load?

    • No
    • Yes (explain why)

Defaults & Configuration

Does this change modify default bot behavior?

    • No
    • Yes (explain why)

If this introduces more advanced or AI-heavy logic:

    • Lightweight mode remains the default
    • More complex behavior is optional and thereby configurable

AI Assistance

Was AI assistance (e.g. ChatGPT or similar tools) used while working on this change?

    • No
    • Yes (explain below)

If yes, please specify:

Copilot CLI to code review


Final Checklist

    • Stability is not compromised
    • Performance impact is understood, tested, and acceptable
    • Added logic complexity is justified and explained
    • Documentation updated if needed

@brighton-chi
Copy link
Contributor

References to "horseman" should be to "horsemen" actually

@kadeshar
Copy link
Collaborator Author

References to "horseman" should be to "horsemen" actually

Fixed that also

@kadeshar kadeshar merged commit a695ac7 into mod-playerbots:test-staging Mar 13, 2026
3 of 9 checks passed
@kadeshar kadeshar deleted the naxxramas-fixes branch March 22, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants