Skip to content

Make BRP builtins utilities parse and parse_some public #22005

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Nilirad wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Make BRP builtins utilities parse and parse_some public #22005

Nilirad wants to merge 1 commit into from
+2 −2

Conversation

@Nilirad
Copy link
Contributor

@Nilirad Nilirad commented Dec 2, 2025

Objective

Builtin BRP methods make constant use of parse and parse_some in parsing utilities. Making them public allows users to use them in their own code.

Solution

Make both functions pub.

Testing

I think testing is not necessary, especially since doc strings do not link anywhere.

@Nilirad Nilirad added C-Usability A targeted quality-of-life change that makes Bevy easier to use A-Networking Sending data between clients, servers and machines labels Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

A-Networking Sending data between clients, servers and machines C-Usability A targeted quality-of-life change that makes Bevy easier to use

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Nilirad