Skip to content

Pr branch 4 e3 va - #15

Open
melsk-r wants to merge 2 commits into
E3V3A:masterfrom
melsk-r:PR-branch-4-E3VA
Open

Pr branch 4 e3 va#15
melsk-r wants to merge 2 commits into
E3V3A:masterfrom
melsk-r:PR-branch-4-E3VA

Conversation

@melsk-r

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

Copy link
Copy Markdown

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

E3V3A commented Oct 21, 2024

Copy link
Copy Markdown
Owner

@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

melsk-r commented Oct 28, 2024

Copy link
Copy Markdown
Author

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