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

Organise workspace crates and consolidate Cargo.toml manifest properties/dependencies #285

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

Swoorup
Copy link
Contributor

@Swoorup Swoorup commented Mar 30, 2024

Simplify hacking on this repo by

  • Allowing to change the version from a single Cargo.toml instead of multiple crates.
  • Single place to update external dependency version instead of multiple cargo.toml's

@Swoorup Swoorup changed the title Organise workspace crates and unify Cargo.toml manifests Organise workspace crates and consolidate Cargo.toml manifests Mar 30, 2024
@Swoorup Swoorup changed the title Organise workspace crates and consolidate Cargo.toml manifests Organise workspace crates and consolidate Cargo.toml manifest properties/dependencies Mar 30, 2024
@Maxxen
Copy link
Member

Maxxen commented Apr 17, 2024

This is def. a good idea, I'll pick this up and solve the conflicts after next release

@Swoorup
Copy link
Contributor Author

Swoorup commented May 1, 2024

Sure, I based few more changes, that fixes quite a far bit of the issues with appender (including adding support of union types and works well with nested types) atop this branch. Would be great to have it in. Let me know if anything is needed.

😃

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.

None yet

2 participants