Skip to content
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.

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

expose parse & parse_some from bevy_remote crate #17732

Open
Utope opened this issue Feb 7, 2025 · 0 comments
Open

expose parse & parse_some from bevy_remote crate #17732

Utope opened this issue Feb 7, 2025 · 0 comments
Labels
C-Feature A new feature, making something new possible S-Needs-Triage This issue needs to be labelled

Comments

@Utope
Copy link

Utope commented Feb 7, 2025

What problem does this solve or what need does it fill?

When writing custom methods with bevy_remote you need to parse the params. The builtin_methods for bevy_remote have the helpers "parse" and "parse_some" which solve this issue. I believe they should be exposed in some way.

What solution would you like?

make parse & parse_some pub. Though they are in mod builtin_methods within bevy_remote so maybe they should be moved for a better public api.

What alternative(s) have you considered?

don't make them public

Additional context

When implementing my custom methods i copy pasted parse and parse_some into my code as they did exactly what I wanted.

@Utope Utope added C-Feature A new feature, making something new possible S-Needs-Triage This issue needs to be labelled labels Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Feature A new feature, making something new possible S-Needs-Triage This issue needs to be labelled
Projects
None yet
Development

No branches or pull requests

1 participant