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

Fixed release trigger #5499

Merged
merged 1 commit into from
Sep 17, 2016
Merged

Fixed release trigger #5499

merged 1 commit into from
Sep 17, 2016

Conversation

Gobberwart
Copy link
Contributor

Short Description:

On TransferPokemon task, the bot randomizes min_free_slot by up to 20 (to simulate human-like behaviour?), so a value of 5 in config could actually be anything from 5 to -15. Bag limit never gets below zero, so bag has very large chance of filling up. Changed so will trigger if space <= 1 if random gives anything equal to or below 0.

Fixes/Resolves/Closes (please use correct syntax):

Bot randomizes min_free_slot by up to 20, so a value of 5 in config
could actually be anything from 5 to -15. Bag limit never gets below
zero, so bag has very large chance of filling up. Changed so will
trigger if space <= 1 if random gives anything equal to or below 0.
@mention-bot
Copy link

@Gobberwart, thanks for your PR! By analyzing the annotation information on this pull request, we identified @Napator, @LitRidl and @alvarogzp to be potential reviewers

@Gobberwart
Copy link
Contributor Author

Very minor change, merging.

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.

2 participants