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

feat!(update/remove): parallelize + use builder pattern #301

Merged
merged 5 commits into from
Jan 4, 2025

Conversation

mrcjkb
Copy link
Member

@mrcjkb mrcjkb commented Jan 2, 2025

Stacked on #300.

@vhyrro I've pulled in the changes from #264, because it keeps getting conflicted.
It's still missing the :all syntax, but I'd suggest you add that in a separate PR 😄

@mrcjkb mrcjkb force-pushed the mj/push-xvosyourorvm branch from fbe6edd to 1b890a6 Compare January 2, 2025 21:49
@mrcjkb mrcjkb force-pushed the mj/push-rtppvxwqtzny branch from f6dfd9a to 6ed86df Compare January 2, 2025 21:49
@mrcjkb mrcjkb force-pushed the mj/push-xvosyourorvm branch from 1b890a6 to 91c0766 Compare January 2, 2025 22:43
@mrcjkb mrcjkb force-pushed the mj/push-rtppvxwqtzny branch from 6ed86df to 8cac23d Compare January 2, 2025 23:44
@mrcjkb mrcjkb force-pushed the mj/push-xvosyourorvm branch from 91c0766 to 3e0d14d Compare January 2, 2025 23:44
@mrcjkb mrcjkb force-pushed the mj/push-rtppvxwqtzny branch from 8cac23d to 2c10bbd Compare January 3, 2025 13:00
@mrcjkb mrcjkb force-pushed the mj/push-xvosyourorvm branch from 3e0d14d to 438d47a Compare January 3, 2025 13:00
Copy link
Contributor

@vhyrro vhyrro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, I'll close my PR after this gets merged :)

@mrcjkb mrcjkb requested a review from teto as a code owner January 4, 2025 18:10
@mrcjkb mrcjkb merged commit 6f345a6 into mj/push-rtppvxwqtzny Jan 4, 2025
13 checks passed
@mrcjkb mrcjkb deleted the mj/push-xvosyourorvm branch January 4, 2025 18:10
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