Skip to content

Fix not calling App::finish and App::cleanup in ScheduleRunnerPlugin#9054

Merged
cart merged 1 commit intobevyengine:mainfrom
B-head:fix-plugin-no-finish
Jul 9, 2023
Merged

Fix not calling App::finish and App::cleanup in ScheduleRunnerPlugin#9054
cart merged 1 commit intobevyengine:mainfrom
B-head:fix-plugin-no-finish

Conversation

@B-head
Copy link
Copy Markdown
Contributor

@B-head B-head commented Jul 6, 2023

This pull request is mutually exclusive with #9066.

Objective

Complete the initialization of the plugin in ScheduleRunnerPlugin.

Solution

Wait for asynchronous tasks to complete, then App::finish and App::cleanup in the runner function.

@B-head B-head marked this pull request as ready for review July 6, 2023 00:52
@james7132 james7132 added the C-Bug An unexpected or incorrect behavior label Jul 6, 2023
@james7132 james7132 requested a review from mockersf July 6, 2023 01:00
@james7132 james7132 added the A-App Bevy apps and plugins label Jul 6, 2023
@mockersf mockersf added this to the 0.11 milestone Jul 6, 2023
@mockersf mockersf 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 Jul 8, 2023
@cart cart added this pull request to the merge queue Jul 9, 2023
Merged via the queue into bevyengine:main with commit f213c14 Jul 9, 2023
@B-head B-head deleted the fix-plugin-no-finish branch July 9, 2023 22:26
B-head added a commit to B-head/bevy that referenced this pull request Jul 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-App Bevy apps and plugins C-Bug An unexpected or incorrect behavior 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.

4 participants