Skip to content

Remove unused regex dep from bevy_render#9613

Merged
alice-i-cecile merged 1 commit intobevyengine:mainfrom
rparrett:remove-regex
Aug 29, 2023
Merged

Remove unused regex dep from bevy_render#9613
alice-i-cecile merged 1 commit intobevyengine:mainfrom
rparrett:remove-regex

Conversation

@rparrett
Copy link
Copy Markdown
Contributor

Objective

As far as I can tell, this is no longer needed since the switch to fancier shader imports via naga_oil.

This shouldn't have any affect on compile times because it's in our tree from naga_oil, tracing-subscriber, and rodio.

@alice-i-cecile alice-i-cecile added A-Rendering Drawing game state to the screen C-Code-Quality A section of code that is hard to understand or change labels Aug 29, 2023
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Aug 29, 2023
@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 Aug 29, 2023
Merged via the queue into bevyengine:main with commit ce2ade2 Aug 29, 2023
rdrpenguin04 pushed a commit to rdrpenguin04/bevy that referenced this pull request Jan 9, 2024
# Objective

As far as I can tell, this is no longer needed since the switch to
fancier shader imports via `naga_oil`.

This shouldn't have any affect on compile times because it's in our tree
from `naga_oil`, `tracing-subscriber`, and `rodio`.
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 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