search

sindresorhus / dot-prop

Get, set, or delete a property from a nested object using a dot path
MIT License
812 stars 69 forks source link

Add `deepKeys()` #94

Closed Richienb closed 2 years ago

Richienb commented 2 years ago

Fixes #93

Richienb commented 2 years ago

The PR is complete, but an AVA bug is causing CI to fail

sindresorhus commented 2 years ago

// @mmkal

sindresorhus commented 2 years ago

But the one that adds it should make sure to also document the mentioned use-cases for it.

Should mention all the use-cases.

sindresorhus commented 2 years ago

Can you fix the merge conflict?

sindresorhus commented 2 years ago

Looks good :)

AuthorProxy commented 1 year ago

@mmkal it doesn't work for empty arrays or objects https://github.com/sindresorhus/dot-prop/issues/104