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

MISC: change all nextUpdates to use one Promise #1069

Merged
merged 9 commits into from
Feb 10, 2024

Conversation

Caldwell-74
Copy link
Contributor

@Caldwell-74 Caldwell-74 commented Jan 27, 2024

partly a duplicate to #1068 but didnt want to fish that one change out of it since i had them all allready done

@Caldwell-74
Copy link
Contributor Author

i changed the sleeves bladeburner task to only create promises when needed instead of for each task by default

@Snarling
Copy link
Collaborator

Due to duplicate types being made, I just made a generic PromisePair type for storing a promise and its resolver. Rebase was just a conflict with corporation imports.

@Snarling Snarling merged commit 8c2a23f into bitburner-official:dev Feb 10, 2024
5 checks passed
@Caldwell-74
Copy link
Contributor Author

i did think about that but wasnt comfy with generics in general but thanks and good job!

Semanual pushed a commit to Semanual/bitburner-src that referenced this pull request Feb 10, 2024
@Caldwell-74 Caldwell-74 deleted the too-many-promises branch February 10, 2024 15:42
antoinedube pushed a commit to antoinedube/bitburner-source that referenced this pull request Mar 17, 2024
G4mingJon4s pushed a commit to G4mingJon4s/bitburner-src that referenced this pull request Mar 23, 2024
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