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

Simplify add_block_batch by making it take PeerInfo instead of WSChiaConnection #16692

Merged

Conversation

AmineKhaldi
Copy link
Contributor

No description provided.

@AmineKhaldi AmineKhaldi added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Cleanup Code cleanup labels Oct 25, 2023
@AmineKhaldi AmineKhaldi self-assigned this Oct 25, 2023
@AmineKhaldi AmineKhaldi marked this pull request as ready for review October 25, 2023 15:54
@AmineKhaldi AmineKhaldi requested a review from a team as a code owner October 25, 2023 15:54
@AmineKhaldi AmineKhaldi changed the title Simplify add_block_batch by making it take PerrInfo instead of WSChiaConnection Simplify add_block_batch by making it take PeerInfo instead of WSChiaConnection Oct 25, 2023
@AmineKhaldi AmineKhaldi force-pushed the simplify_add_block_batch_peer_info branch from 235a5f2 to d4a9ea7 Compare October 25, 2023 16:01
@github-actions github-actions bot added the merge_conflict Branch has conflicts that prevent merge to main label Oct 25, 2023
@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

arvidn
arvidn previously approved these changes Oct 25, 2023
@AmineKhaldi AmineKhaldi force-pushed the simplify_add_block_batch_peer_info branch from d4a9ea7 to a666bbf Compare October 26, 2023 12:04
@github-actions github-actions bot removed the merge_conflict Branch has conflicts that prevent merge to main label Oct 26, 2023
@github-actions
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@arvidn arvidn added the ready_to_merge Submitter and reviewers think this is ready label Oct 26, 2023
@cmmarslender cmmarslender merged commit a420d8e into Chia-Network:main Oct 26, 2023
247 of 248 checks passed
@arvidn arvidn mentioned this pull request Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Cleanup Code cleanup ready_to_merge Submitter and reviewers think this is ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants