Skip to content

Added derive for arbitrary#2193

Closed
LucaCappelletti94 wants to merge 1 commit intoapache:mainfrom
LucaCappelletti94:arbitrary
Closed

Added derive for arbitrary#2193
LucaCappelletti94 wants to merge 1 commit intoapache:mainfrom
LucaCappelletti94:arbitrary

Conversation

@LucaCappelletti94
Copy link
Copy Markdown
Contributor

Adds Arbitrary derives to all AST structs and enums for property-based testing and fuzzing.

Changes

  • Added arbitrary as optional dependency with feature gating
  • Derived Arbitrary on all public AST types in ast
  • Added Arbitrary to core types (Token, Span, Location, etc.)
  • Excluded display helpers and iterators

Note: BigDecimal (used in Value::Number) lacks Arbitrary support. PR submitted: akubera/bigdecimal-rs#163

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

Thank you for your contribution. Unfortunately, this pull request is stale because it has been open 60 days with no activity. Please remove the stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label Apr 6, 2026
@github-actions github-actions bot closed this Apr 13, 2026
@LucaCappelletti94
Copy link
Copy Markdown
Contributor Author

This type of GitHub Actions is not positive IMHO. This PR is blocked by another PR uphill, and closing it does not make the problem go away.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant