search

jorinvo / edn-data

EDN parser and generator that works with plain JS data, with support for TS and node streams
MIT License
96 stars 5 forks source link

Feat: obectsKeysAs feature #20

Closed sielay closed 9 months ago

jorinvo commented 9 months ago

Hey there, happy to see people contributing! I get the idea from the description in the code. I don't mind adding this. It seems like a more useful result than without the option. For your particular use case have you considered setting mapAs to a different value?

jorinvo commented 9 months ago

Published the change as version 1.1.0 now: https://www.npmjs.com/package/edn-data

sielay commented 9 months ago

Hey there, happy to see people contributing! I get the idea from the description in the code. I don't mind adding this. It seems like a more useful result than without the option. For your particular use case have you considered setting mapAs to a different value?

Hi. I played with mapAs but couldn't find the format that would work best for us in that case.

Kudos for the fast review and merge.