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

Update: Cargo build script and struct impls #1

Merged
merged 5 commits into from
Jan 27, 2023
Merged

Update: Cargo build script and struct impls #1

merged 5 commits into from
Jan 27, 2023

Conversation

michaelgrigoryan25
Copy link

@michaelgrigoryan25 michaelgrigoryan25 commented Jan 27, 2023

  • Implemented std::default::Default for Database and cleaned up the implementation of Database::new.
  • Updated Cargo build script. The bindings will be generated on the fly and will be put in OUT_DIR. Bindings will be linked directly to the crate, without the need to keep them tracked in git.
  • Removed build = ... from Cargo.toml. Cargo finds the build script automatically.

- Implemented std::default::Default for Database and cleaned up the
  implementation of Database::new.
- Updated Cargo build script. The bindings will be generated on the
  fly and will be put in `OUT_DIR`. Bindings will be linked directly
  to the crate, without the need to keep them tracked in git.
@chungquantin chungquantin merged commit de1ba92 into chungquantin:dev Jan 27, 2023
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.

2 participants