Gino Valente
ginovalente.bsky.social
Gino Valente
@ginovalente.bsky.social
💻👾☕️
Bevy’s great! If you already know Rust or want to learn Rust, it’s a really fun code-first framework to use
April 18, 2025 at 5:32 AM
Remember kids: document your code changes!
March 30, 2025 at 4:03 AM
Alright, that was pretty straightforward. This is the reason I try to write detailed PR descriptions. I know my memory would not have served me well had I not left good notes for myself 😅

github.com/bevyengine/b...
Add release note for function overloading by MrGVSV · Pull Request #2042 · bevyengine/bevy-website
Resolves #1969
github.com
March 30, 2025 at 4:03 AM
I still think it's pretty cool I managed to get this working at all

```rust
// Allows adding two `i32`, `u32`, or `f32` numbers
let reflect_add = add::<i32>
.into_function()
.with_overload(add::<u32>)
.with_overload(add::<f32>);
```
March 30, 2025 at 3:36 AM
This was merged towards the beginning of the 0.16 cycle back in December so I actually forgot about it up until now lol
March 30, 2025 at 3:33 AM
Actually, we can do `clone_value` -> `to_dynamic` and `clone_dynamic` -> `to_dynamic_***`.

This should allow us to properly deprecate all the replaced methods rather than just removing them outright!
March 12, 2025 at 12:52 AM
So I think I can try renaming the conflicting methods on the reflection subtraits to things like `to_dynamic_struct`.

That's a little unfortunate because I wanted to make this a non-breaking change.

Alternatively, I could try to find a better name for `clone_value`.
March 12, 2025 at 12:44 AM
I think the next thing I'd like to work on is some more minor type data and/or macro improvements since it would at least be nice if those coincided with the `reflect_clone` PR landing in 0.16 as well

github.com/bevyengine/b...
bevy_reflect: Add `ReflectClone` type data by MrGVSV · Pull Request #18211 · bevyengine/bevy
Objective This PR is a partial followup to #13432, but is not based on that branch as it also works as a completely standalone change. The goal of this PR is to add reflection type data for the Clo...
github.com
March 10, 2025 at 6:39 AM
Okay hopefully that does it (accidentally imported `bevy_reflect` instead of `bevy::reflect` in an example).

Normally, I'd wait for all of CI to pass but my son has an early appointment tomorrow. I enjoyed finally having an hour to work on some #bevy code though 😭
March 10, 2025 at 6:37 AM
Actually, CI is failing. Hold please!
March 10, 2025 at 6:35 AM
Aaaand we're done! Just goes to show that sometimes a PR looks scarier to rebase than it actually is 😄
March 10, 2025 at 6:34 AM
Final commits were super easy: first was just adding a compile fail test (only had conflicts due to a change I made in the first commit) and the second was just a tiny bug fix.
March 10, 2025 at 6:33 AM
First commit down! Mostly just renames that git couldn't handle. But seems there have been quite a few new `FromType` usages added to the repo since almost a year ago, including a dedicated type data example!

My usual checks are passing, so moving on!
March 10, 2025 at 6:24 AM
I try not to squash as I like to break my commits into manageable parts with a (somewhat) clear timeline. For large PRs this preference can really be a pain.

Thankfully, this PR only has 3 commits :)
March 10, 2025 at 5:51 AM
This PR should also fix a mild annoyance of mine that `FromType` should really be given a name more apparently related to "type data"
March 10, 2025 at 5:46 AM