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

Added a new cli option, --delay-range. The in-stock check will use a … #399

Open
wants to merge 5 commits into
base: development
Choose a base branch
from

Conversation

caustiq
Copy link

@caustiq caustiq commented Dec 8, 2020

…random delay within the specified range.

The default value for this is (None, None) because I needed a way to check for the existence of, and utilize, the main delay option.
Unless we specify a default, the click library complains. Might be a bug in click.

…random delay within the specified range.

The default value for this is (None, None) because I needed a way to check for the existence of, and utilize, the main `delay` option.
Unless we specify a default, the click library complains. Might be a bug in click.
@caustiq
Copy link
Author

caustiq commented Dec 8, 2020

Anyways, the default ends up being (3.0, 3.0)

@ghost
Copy link

ghost commented Dec 8, 2020

Why is this better than the existing --delay?

@caustiq
Copy link
Author

caustiq commented Dec 8, 2020

Why is this better than the existing --delay?

No evidence that it's better. And it doesn't replace --delay.

It likely wouldn't make you look any less of a bot overall, but the possibility that it'd appear less bot-like might be greater than zero based on some of the discord users' discussions.

I don't particularly agree as there's no evidence but I figured it may be worth experimenting with to see if it helps out any with doggos.

@jay-babu
Copy link
Contributor

+1 this change

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