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.

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

bevy_reflect: Ignored field order #6511

Closed
wants to merge 5 commits into from
Closed

bevy_reflect: Ignored field order #6511

wants to merge 5 commits into from

Commits on Jan 2, 2023

  1. Error on bad #[reflect(ignore)] ordering

    @MrGVSV
    MrGVSV committed Jan 2, 2023
    Configuration menu
    Copy the full SHA
    56a4c9f View commit details
    Browse the repository at this point in the history

  2. Update crates/bevy_reflect/bevy_reflect_derive/src/field_attributes.rs 

    Co-authored-by: Nicola Papale <nicopap@users.noreply.github.com>
    @MrGVSV @nicopap
    MrGVSV and nicopap committed Jan 2, 2023
    Configuration menu
    Copy the full SHA
    bf3e62d View commit details
    Browse the repository at this point in the history

  3. Convert if-let to match

    @MrGVSV
    MrGVSV committed Jan 2, 2023
    Configuration menu
    Copy the full SHA
    57dc883 View commit details
    Browse the repository at this point in the history

  4. Add compile fail tests for Reflect field attributes

    @MrGVSV
    MrGVSV committed Jan 2, 2023
    Configuration menu
    Copy the full SHA
    f591183 View commit details
    Browse the repository at this point in the history

  5. Move combine_error to utility module

    @MrGVSV
    MrGVSV committed Jan 2, 2023
    Configuration menu
    Copy the full SHA
    a2b1f94 View commit details
    Browse the repository at this point in the history