Skip to content

Move uniform out of extract_component#23372

Merged
alice-i-cecile merged 1 commit intobevyengine:mainfrom
Zeophlite:render-uniform
Mar 15, 2026
Merged

Move uniform out of extract_component#23372
alice-i-cecile merged 1 commit intobevyengine:mainfrom
Zeophlite:render-uniform

Conversation

@Zeophlite
Copy link
Copy Markdown
Contributor

@Zeophlite Zeophlite commented Mar 15, 2026

Objective

Solution

  • Move uniforms out of extract_component.rs

Testing

  • CI

@Zeophlite Zeophlite added D-Trivial Nice and easy! A great choice to get started with Bevy A-Rendering Drawing game state to the screen S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Mar 15, 2026
@github-project-automation github-project-automation bot moved this to Needs SME Triage in Rendering Mar 15, 2026
@alice-i-cecile alice-i-cecile added C-Code-Quality A section of code that is hard to understand or change S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Mar 15, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Mar 15, 2026
Merged via the queue into bevyengine:main with commit 9add6e1 Mar 15, 2026
42 checks passed
@github-project-automation github-project-automation bot moved this from Needs SME Triage to Done in Rendering Mar 15, 2026
splo pushed a commit to splo/bevy that referenced this pull request Mar 31, 2026
# Objective

- Uniform handling is an application of extract components, not part of
its implementation
- Step towards bevyengine#22852

## Solution

- Move uniforms out of `extract_component.rs`

## Testing

- CI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Rendering Drawing game state to the screen C-Code-Quality A section of code that is hard to understand or change D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants