Skip to content

Remove deprecated generate_view_layouts function#23331

Closed
tauanbinato wants to merge 1 commit intobevyengine:mainfrom
tauanbinato:fix/issue-17537
Closed

Remove deprecated generate_view_layouts function#23331
tauanbinato wants to merge 1 commit intobevyengine:mainfrom
tauanbinato:fix/issue-17537

Conversation

@tauanbinato
Copy link
Contributor

@tauanbinato tauanbinato commented Mar 12, 2026

Objective

Solution

  • Remove the deprecated generate_view_layouts function from bevy_pbr::render::mesh_view_bindings
  • Update the doc comment on MeshPipelineViewLayoutKey to reference layout_entries instead of the removed function

Testing

  • cargo check -p bevy_pbr passes cleanly
  • No other references to generate_view_layouts exist in the codebase

Migration Guide

  • generate_view_layouts has been removed. Use layout_entries instead, as indicated by the deprecation notice added in 0.19.0.

Copy link
Contributor

@ChristopherBiscardi ChristopherBiscardi left a comment

Choose a reason for hiding this comment

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

0.19 hasn't been released yet, so this should not be removed until the deprecation can go out for users to see

@tauanbinato
Copy link
Contributor Author

0.19 hasn't been released yet, so this should not be removed until the deprecation can go out for users to see

No problem, should I close the pr? Thanks

@ChristopherBiscardi
Copy link
Contributor

I'll leave that up to you. From what I know, I don't see this being merged until at least 0.19's release candidate has started, and I'm not sure when the deprecation is intended to be finalized with a removal. Usually its at least one version. We could probably do all the 0.19 deprecation removals once 0.20's development cycle kicks off.

So if you leave it open it'll be open for a couple weeks at least.

@alice-i-cecile
Copy link
Member

We'll get rid of all of the deprecations in one pass later.

@tauanbinato tauanbinato deleted the fix/issue-17537 branch March 14, 2026 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove bevy_pbr::generate_view_layouts

3 participants