Skip to content

[Merged by Bors] - Reduce visibility of various types and fields #2690

Closed
bjorn3 wants to merge 1 commit intobevyengine:mainfrom
bjorn3:ecs_improvements2
Closed

[Merged by Bors] - Reduce visibility of various types and fields #2690
bjorn3 wants to merge 1 commit intobevyengine:mainfrom
bjorn3:ecs_improvements2

Conversation

@bjorn3
Copy link
Contributor

@bjorn3 bjorn3 commented Aug 19, 2021

See the individual commits.

@github-actions github-actions bot added the S-Needs-Triage This issue needs to be labelled label Aug 19, 2021
@bjorn3 bjorn3 added A-ECS Entities, components, systems, and events C-Code-Quality A section of code that is hard to understand or change and removed S-Needs-Triage This issue needs to be labelled labels Aug 19, 2021
Copy link
Member

@DJMcNab DJMcNab left a comment

Choose a reason for hiding this comment

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

Makes sense, although I might ask you not to include the resolve_unknown commit because of my pr which does something similar but keeps the method.

to match the visibility of the containing module or type
@bjorn3 bjorn3 force-pushed the ecs_improvements2 branch from 92a3fc6 to d092e81 Compare August 19, 2021 16:22
@bjorn3 bjorn3 changed the title Couple of small ecs cleanups Reduce visibility of various types and fields Aug 19, 2021
@mockersf mockersf 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 19, 2021
@cart
Copy link
Member

cart commented Aug 19, 2021

bors r+

bors bot pushed a commit that referenced this pull request Aug 19, 2021
@bors bors bot changed the title Reduce visibility of various types and fields [Merged by Bors] - Reduce visibility of various types and fields Aug 19, 2021
@bors bors bot closed this Aug 19, 2021
@bjorn3 bjorn3 deleted the ecs_improvements2 branch August 19, 2021 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ECS Entities, components, systems, and events 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.

4 participants