search

borkdude / rewrite-edn

Utility lib on top of rewrite-clj with common operations to update EDN while preserving whitespace and comments.
Eclipse Public License 1.0
84 stars 14 forks source link

keys function #14

Closed witek closed 2 years ago

borkdude commented 2 years ago

Thanks!

witek commented 2 years ago

Suddenly I am not sure if I have done the right thing in this pull request.

The keys function returns the sexpressions for the keys. But shouldn't it return the nodes for the keys instead?

borkdude commented 2 years ago

@witek That would maybe be the better behavior since you can always call sexpr on those, but not vice versa.

witek commented 2 years ago

I will make another pull request...

borkdude commented 2 years ago

Thank you!