Skip to content

Fix minimal plugins in ci#12370

Merged
alice-i-cecile merged 2 commits intobevyengine:mainfrom
mockersf:fix-minimal-plugins-in-ci
Mar 7, 2024
Merged

Fix minimal plugins in ci#12370
alice-i-cecile merged 2 commits intobevyengine:mainfrom
mockersf:fix-minimal-plugins-in-ci

Conversation

@mockersf
Copy link
Member

@mockersf mockersf commented Mar 7, 2024

Objective

Solution

  • include DevToolsPlugin in MinimalPlugins

@mockersf mockersf added C-Bug An unexpected or incorrect behavior A-Build-System Related to build systems or continuous integration labels Mar 7, 2024
@alice-i-cecile
Copy link
Member

@matiqo15 can I get your review here?

@alice-i-cecile alice-i-cecile requested a review from matiqo15 March 7, 2024 22:15
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Mar 7, 2024
@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 Mar 7, 2024
Merged via the queue into bevyengine:main with commit 0baedcf Mar 7, 2024
github-merge-queue bot pushed a commit that referenced this pull request Mar 8, 2024
…le (#12385)

# Objective

- After #12370, ci testing with minimal plugins doesn't hang but it
crash as the resource `ScreenshotManager` doesn't exist

## Solution

- Check if the resource exists
spectria-limina pushed a commit to spectria-limina/bevy that referenced this pull request Mar 9, 2024
# Objective

- bevyengine#11341 broke running code using `MinimalPlugins` in CI

## Solution

- include `DevToolsPlugin` in `MinimalPlugins`
spectria-limina pushed a commit to spectria-limina/bevy that referenced this pull request Mar 9, 2024
…le (bevyengine#12385)

# Objective

- After bevyengine#12370, ci testing with minimal plugins doesn't hang but it
crash as the resource `ScreenshotManager` doesn't exist

## Solution

- Check if the resource exists
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Build-System Related to build systems or continuous integration 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.

3 participants