Skip to content

[Merged by Bors] - IntoIterator for EventReader#7720

Closed
Aceeri wants to merge 1 commit intobevyengine:mainfrom
Aceeri:event-reader-into-iter
Closed

[Merged by Bors] - IntoIterator for EventReader#7720
Aceeri wants to merge 1 commit intobevyengine:mainfrom
Aceeri:event-reader-into-iter

Conversation

@Aceeri
Copy link
Member

@Aceeri Aceeri commented Feb 16, 2023

Objective

Continuation of #5719
Now that we have a definable type for the iterator.


Changelog

  • Implemented IntoIterator for EventReader so you can now do &mut reader instead of reader.iter() for events.

@Aceeri Aceeri added A-ECS Entities, components, systems, and events C-Usability A targeted quality-of-life change that makes Bevy easier to use labels Feb 16, 2023
@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Feb 16, 2023
@alice-i-cecile
Copy link
Member

Merging as trivial.

bors r+

bors bot pushed a commit that referenced this pull request Feb 17, 2023
# Objective
Continuation of #5719
Now that we have a definable type for the iterator.

---

## Changelog
- Implemented IntoIterator for EventReader so you can now do `&mut reader` instead of `reader.iter()` for events.
@bors
Copy link
Contributor

bors bot commented Feb 17, 2023

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Feb 17, 2023
# Objective
Continuation of #5719
Now that we have a definable type for the iterator.

---

## Changelog
- Implemented IntoIterator for EventReader so you can now do `&mut reader` instead of `reader.iter()` for events.
Copy link
Member

@alice-i-cecile alice-i-cecile left a comment

Choose a reason for hiding this comment

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

bors r+

@bors
Copy link
Contributor

bors bot commented Feb 17, 2023

Already running a review

@bors bors bot changed the title IntoIterator for EventReader [Merged by Bors] - IntoIterator for EventReader Feb 17, 2023
@bors bors bot closed this Feb 17, 2023
myreprise1 pushed a commit to myreprise1/bevy that referenced this pull request Feb 18, 2023
# Objective
Continuation of bevyengine#5719
Now that we have a definable type for the iterator.

---

## Changelog
- Implemented IntoIterator for EventReader so you can now do `&mut reader` instead of `reader.iter()` for events.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ECS Entities, components, systems, and events C-Usability A targeted quality-of-life change that makes Bevy easier to use S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants