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

Pr branch 4 e3 va #15

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Pr branch 4 e3 va #15

wants to merge 2 commits into from

Conversation

melsk-r
Copy link

@melsk-r melsk-r commented Jul 3, 2024

Some corrections in the code.

Not within this PR:

  • The code on line 257 n += 1 did not work with me so I replaced this with n = n + 1.
  • I took some measures to prevent secondairy rate limits. Unfortunately creating a personal authentication token did not solve that problem. As a workaround I placed sleep(300) commands within the code and ran this script in batches of 25 issues.

melsk-r added 2 commits July 3, 2024 11:02
Some corrections in the code.

The code on line 257 'n += 1 did not work with me so I replaced this with n = n + 1.
@E3V3A
Copy link
Owner

E3V3A commented Oct 21, 2024

@melsk-r
Hi Robert,
Thanks for your PR! I haven't used this for nearly 3 years, so surprised if it still works.
However, that the assignment n += 1 doesn't work, sounds very strange!

@melsk-r
Copy link
Author

melsk-r commented Oct 28, 2024

I know, however it really was the case. Maybe something else caused it. I leave it up to you to change the code or not. It's not a problem for me if you don't.

Greetings

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