tsibley / App-cpangrep

Grep CPAN from the command-line using grep.cpan.me
Other
4 stars 1 forks source link

change backend from JSON to JSON::MaybeXS #8

Closed karenetheridge closed 4 years ago

karenetheridge commented 4 years ago

..for providing the least insane conduit to JSON decoding with least surprises to the user

(I didn't update META.json or Makefile.PL as milla updates those from cpanfile.)

shadowcat-mst commented 4 years ago

Please please please please please please.

Use of JSON.pm drags in 5.005 compat crazy and JSON::XS, which means Marc Lehmann, which means everything is made of urine and you can't even find the cat.

We wrote JSON::MaybeXS to never have to deal with those problems again and would appreciate your adopting the anti-footgun device.

tsibley commented 4 years ago

Hi ether and mst! Thanks for the PR. All my newer stuff uses JSON::MaybeXS, so I’m happy to merge this. Would either of you be up for cutting a release?