-
Notifications
You must be signed in to change notification settings - Fork 61
Embezzler lucky #2367
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
Merged
Merged
Embezzler lucky #2367
Changes from 8 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
1405d59
Support for lucky adventures for embezzlers
Shiverwarp 355bbcb
Run the fights
Shiverwarp f5c78e4
Change where we fight embezzlers if not targetingMeat
Shiverwarp d67480c
Run fights after buffing
Shiverwarp 373d239
Check for lucky in meatTargetOutfit
Shiverwarp 53a0202
Move check earlier in ready:
Shiverwarp a491fd7
Pretty up meatTargetOutfit call
Shiverwarp 8896926
Add turns, consolidate nonbarf lucky
Shiverwarp 9f5f80b
copyTargetCount() related changes
Shiverwarp dc726b9
top level variable classic
Shiverwarp 1866836
Using a function to fix undefined error
Shiverwarp 997c579
Implement highMeatMonsterTargets()
Shiverwarp 4723b91
Put AlternateTask in engine
Shiverwarp 6487d41
type fixes for embezzlers
spaghetti-squash 94e13fb
pull out garboStrategy into combatStrategy.ts
Shiverwarp 8e156d7
Check for exclusion first
Shiverwarp 40b2500
Avoid scepter sometimes
Shiverwarp bbb4f25
Merge branch 'main' into embezzler-lucky
Ignose 38a1b83
Merge branch 'main' into embezzler-lucky
spaghetti-squash ead758c
only call highMeatMonsterCount once for robort
Shiverwarp d4cc632
Merge branch 'embezzler-lucky' of https://github.com/loathers/garbage…
Shiverwarp 69ff33c
Merge branch 'main' of https://github.com/loathers/garbage-collector …
Shiverwarp 1d6eef9
new GarboStrategy import
Shiverwarp File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this worth having two separate functions (or a boolean arg) for? is everywhere we call
copyTargetCount
a place where we want to include embezzlers? even if you're fighting, say, roaches?