Skip to content

Update increment_change_tick to return a strongly-typed Tick#8295

Merged
james7132 merged 1 commit intobevyengine:mainfrom
joseph-gio:increment-tick
Apr 3, 2023
Merged

Update increment_change_tick to return a strongly-typed Tick#8295
james7132 merged 1 commit intobevyengine:mainfrom
joseph-gio:increment-tick

Conversation

@joseph-gio
Copy link
Member

Objective

While migrating the engine to use the Tick type in #7905, I forgot to update UnsafeWorldCell::increment_change_tick.

Solution

Update the function.


Changelog

  • The function UnsafeWorldCell::increment_change_tick is now strongly-typed, returning a value of type Tick instead of a raw u32.

Migration Guide

The function UnsafeWorldCell::increment_change_tick is now strongly-typed, returning a value of type Tick instead of a raw u32.

@joseph-gio joseph-gio added A-ECS Entities, components, systems, and events C-Usability A targeted quality-of-life change that makes Bevy easier to use M-Migration-Guide A breaking change to Bevy's public API that needs to be noted in a migration guide labels Apr 3, 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 Apr 3, 2023
@james7132 james7132 added this pull request to the merge queue Apr 3, 2023
Merged via the queue into bevyengine:main with commit 086db6d Apr 3, 2023
@joseph-gio joseph-gio deleted the increment-tick branch April 3, 2023 18:47
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 M-Migration-Guide A breaking change to Bevy's public API that needs to be noted in a migration guide 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.

3 participants