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.

Sign up for GitHub

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

Jump to bottom

Cleanup request queue #826

Merged

Koenkk merged 3 commits into Koenkk:master from slugzero:request-queue-cleanup

Dec 10, 2023

Contributor

slugzero commented Dec 9, 2023

Just some cleanup:

fix review findings from previous PR Improve Request Queue Behavior #2 #700 (comment)
endpoint.ts has grown quite big, so I moved everything related to the request queue to a separate class and file
The rest is adapting the tests to the new data structures and fixing some flakiness in the tests which made them fail due to changed timing.

slugzero added 3 commits

December 8, 2023 11:18


          Cleanup: variable rename

a671011


          Cleanup: move requestQueue to separate file, rename variables

f4cdfe2


          Adapt data types in tests

4b6fe38

Koenkk merged commit db1545b into Koenkk:master

1 check passed

Owner

Koenkk commented Dec 10, 2023

Thanks!

Koenkk mentioned this pull request

chore(master): release 0.26.1 #825

Merged

slugzero deleted the request-queue-cleanup branch

December 16, 2023 21:07

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet