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

fix(stackable-versioned): Validation error reporting #842

Merged
merged 7 commits into from
Aug 21, 2024
Merged

fix(stackable-versioned): Validation error reporting #842

merged 7 commits into from
Aug 21, 2024

Commits on Aug 21, 2024

  1. fix: Report variant name validation error at the correct span

    @Techassi
    Techassi committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    afcef1b View commit details
    Browse the repository at this point in the history

  2. fix: Remove leading underscore for variant not using PascalCase

    @Techassi
    Techassi committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    7d42185 View commit details
    Browse the repository at this point in the history

  3. chore: Update changelog

    @Techassi
    Techassi committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    4ccec93 View commit details
    Browse the repository at this point in the history

  4. chore: Update PR link in changelog

    @Techassi
    Techassi committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    403a649 View commit details
    Browse the repository at this point in the history

  5. style: Update format_ident call

    @Techassi
    Techassi committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    828200e View commit details
    Browse the repository at this point in the history

  6. chore: Apply suggestion 

    Co-authored-by: Nick <[email protected]>
    @Techassi @NickLarsenNZ
    Techassi and NickLarsenNZ authored Aug 21, 2024
    Configuration menu
    Copy the full SHA
    9d98236 View commit details
    Browse the repository at this point in the history

  7. chore: Apply suggestions 

    This snuck in because rust-analyzer doesn't remove trailing commas the same way it does for the format! macro.

Co-authored-by: Nick <[email protected]>
    @Techassi @NickLarsenNZ
    Techassi and NickLarsenNZ authored Aug 21, 2024
    Configuration menu
    Copy the full SHA
    87cc5be View commit details
    Browse the repository at this point in the history