Skip to content

reject nil/empty keys and nil values#99

Merged
erikgrinaker merged 7 commits intomasterfrom
erik/reject-empty-keys
May 19, 2020
Merged

reject nil/empty keys and nil values#99
erikgrinaker merged 7 commits intomasterfrom
erik/reject-empty-keys

Conversation

@erikgrinaker
Copy link
Contributor

Fixes #2.

@erikgrinaker erikgrinaker added T:breaking Type: Breaking S:automerge Status: Auto merge a pull request labels May 18, 2020
@erikgrinaker erikgrinaker requested a review from melekes May 18, 2020 15:16
@erikgrinaker erikgrinaker requested a review from tessr as a code owner May 18, 2020 15:16
@erikgrinaker erikgrinaker self-assigned this May 18, 2020
@erikgrinaker erikgrinaker removed the S:automerge Status: Auto merge a pull request label May 18, 2020
@erikgrinaker erikgrinaker merged commit 7b7076b into master May 19, 2020
@erikgrinaker erikgrinaker deleted the erik/reject-empty-keys branch May 19, 2020 10:22
Copy link
Contributor

@melekes melekes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

nddeluca referenced this pull request in Kava-Labs/tm-db Jan 8, 2024
* Prepare changelog

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Fix date in changelog summary

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Update CHANGELOG.md

Co-authored-by: Andy Nogueira <me@andynogueira.dev>

---------

Signed-off-by: Thane Thomson <connect@thanethomson.com>
Co-authored-by: Andy Nogueira <me@andynogueira.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T:breaking Type: Breaking

Projects

None yet

Development

Successfully merging this pull request may close these issues.

libs/db: disallow empty keys

3 participants