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.

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

Optional serde support #275

Merged
merged 1 commit into from
Feb 18, 2025
Merged

Optional serde support #275

merged 1 commit into from
Feb 18, 2025

Conversation

chrjabs
Copy link
Owner

@chrjabs chrjabs commented Feb 18, 2025

Description of the Contribution

Implement serde::Serialize and serde::Deserialize behind a serde feature.

resolves #270
closes #271

PR Checklist

  • I read and agree to CONTRIBUTING.md
  • I have formatted my code with rustfmt / cargo fmt --all
  • Commits are named following conventional commits
  • I have added documentation for new features
  • The test suite still passes on this PR
  • I have added tests for new features / tests that would have caught the bug this PR fixes (please explain if not)
  • If this PR contains breaking changes, it is against the next-major branch, not against main

@chrjabs chrjabs merged commit cf4fd62 into main Feb 18, 2025
52 checks passed
@chrjabs chrjabs deleted the serde branch February 18, 2025 14:40
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.

Add serde support
1 participant