vincent-herlemont / native_db

Drop-in embedded database in Rust
MIT License
433 stars 17 forks source link

Add `export_keys` optional param to `native_db` macro #246

Open Grimeh opened 1 day ago

Grimeh commented 1 day ago

As discussed in #220, this PR adds an optional export_keys boolean param to the native_db macro which allows for table keys to be made public and thus externally accessible from the crate.

I have left a TODO in tests/macro_def/export_keys_attribute.rs:5 to note that this functionality can't be fully tested without somehow testing from an external crate. I could just remove that TODO if you're happy with how it is, not sure what your thoughts on the matter are.

vincent-herlemont commented 12 hours ago

@Grimeh Thank you for your PR!

vincent-herlemont commented 12 hours ago

@Grimeh Can you rename your commits following the conventional commits standard?