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

CHIA-826 Simplify get_items_by_coin_id and make it return an iterator #18255

Merged

Conversation

AmineKhaldi
Copy link
Contributor

We don't need to use the connection context manager as there is nothing to commit or roll back.

@AmineKhaldi AmineKhaldi added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Cleanup Code cleanup labels Jun 26, 2024
@AmineKhaldi AmineKhaldi self-assigned this Jun 26, 2024
@AmineKhaldi AmineKhaldi marked this pull request as ready for review June 28, 2024 18:19
@AmineKhaldi AmineKhaldi requested a review from a team as a code owner June 28, 2024 18:19
@Starttoaster Starttoaster merged commit 8141fe4 into Chia-Network:main Jul 3, 2024
367 checks passed
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants