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

feat: move serde serialization behind the serialize feature flag #21

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lavafroth
Copy link

@lavafroth lavafroth commented May 30, 2024

Fixes #19

Changes

  • A new feature flag called serialize is created inside Cargo.toml
  • Structures and enums implement serialization and deserialization only when the feature is enabled
  • This feature flag is enabled by default since the main CLI requires reading serialized data
  • Upstream libraries can add default-features = false in their Cargo.toml while adding poasta as a dependency which will remove serialization and deserialization capabilities and gate the save_graph and load_graph functions

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.

Make serde serialization support optional and behind a cargo feature flag
1 participant