Skip to content

Remove paste from definition rule composition - #136

Open
adampetro wants to merge 1 commit into
mainfrom
ap.remove-paste
Open

Remove paste from definition rule composition#136
adampetro wants to merge 1 commit into
mainfrom
ap.remove-paste

Conversation

@adampetro

@adampetro adampetro commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

This change makes definition rules and visitors more consistent with the executable rules and visitors. Both validation surfaces now use the same tuple composition and construction patterns.

  • replace combine_definition_rules! with tuple-based rule composition
  • use seq_macro implementations for definition rule and visitor tuples
  • move definition rule construction to Visitor to align validation surfaces
  • remove the unused iterator macros and the paste dependency

Testing

  • cargo check --workspace --all-targets
  • cargo test -p bluejay-validator
  • cargo test -p bluejay-validator --test definition_integration_test
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cargo fmt --all -- --check

Closes #134

Assisted-By: devx/5ade7443-148c-4a48-928b-d5abe80b0212
@adampetro
adampetro marked this pull request as ready for review August 21, 2026 19:03
@adampetro
adampetro requested a review from swalkinshaw August 21, 2026 19:05
@amnn amnn mentioned this pull request Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

advisory(RUSTSEC-2024-0436): paste dependency unmaintained

1 participant