Remove deprecated generate_view_layouts function#23331
Remove deprecated generate_view_layouts function#23331tauanbinato wants to merge 1 commit intobevyengine:mainfrom
generate_view_layouts function#23331Conversation
ChristopherBiscardi
left a comment
There was a problem hiding this comment.
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 |
|
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. |
|
We'll get rid of all of the deprecations in one pass later. |
Objective
bevy_pbr::generate_view_layouts#17537generate_view_layoutsfunction was deprecated in deprecated generate_view_layouts in favor of layout_entries (#17537) #17714 (0.19.0) in favor oflayout_entries. It has no internal callers and can now be fully removed.Solution
generate_view_layoutsfunction frombevy_pbr::render::mesh_view_bindingsMeshPipelineViewLayoutKeyto referencelayout_entriesinstead of the removed functionTesting
cargo check -p bevy_pbrpasses cleanlygenerate_view_layoutsexist in the codebaseMigration Guide
generate_view_layoutshas been removed. Uselayout_entriesinstead, as indicated by the deprecation notice added in 0.19.0.