Zulu-Inuoe / jzon

A correct and safe(er) JSON RFC 8259 reader/writer with sane defaults.
MIT License
151 stars 14 forks source link

[FR] Add to `quicklisp` #10

Closed NightMachinery closed 1 year ago

Zulu-Inuoe commented 3 years ago

Hi @NightMachinary . I appreciate the support, unfortunately I'd like to avoid putting this on Quicklisp until it's more complete. At a minimum I wish to have:

  1. A roundtripping reader/writer (output JSON exactly as it was read)
  2. Object coercion mechanism for reading (eg read JSON as a CL class/structure)

Until these are done, I don't want to pollute QL with yet another JSON library.

fukamachi commented 3 years ago

+1 for adding to Quicklisp. IMHO, jzon is already the best JSON library for rendering JSON views in web app. So, I'd like to switch from Jonathan in my libraries, but can't because jzon is not in Quicklisp.

Zulu-Inuoe commented 3 years ago

@fukamachi Thanks for the support. Unfortunately I haven't been doing much programming in my free time as of late. But if you think the library is useful as-is, then it's fair to add it to quicklisp, and I can add my other desired features over time as I can.

I just don't want to continue to contribute to the 100 JSON library meme on Quicklisp unless I offer some substantial benefit

Valera commented 3 years ago

@Zulu-Inuoe I think that jzon is a new generation JSON parser taking into account advantages and disadvantages of previous generation of parsers. And it has full coverage of JSON standard verified by third-party test suite. This is already good reasons for releasing v1 on quicklisp.

K1D77A commented 2 years ago

Yes, can you please add this to quicklisp, the pretty print feature is exceptionally useful :)

Shinmera commented 2 years ago

Throwing my hat onto the pile. I'd like to start using this.

phoe commented 2 years ago

Pretty please?

Zulu-Inuoe commented 2 years ago

Oh my gosh. Let me see what I can do this weekend and I'll add v1 on Monday no matter what

Zulu-Inuoe commented 2 years ago

haha jk did I say that weekend? I meant this one.

In all seriousness I've been doing some work and thinking over the week and will definitely have something solid here soon. Thanks all

phoe commented 2 years ago

fbn7krvnq8381

Zulu-Inuoe commented 2 years ago

fbn7krvnq8381

Unfortunately I am a busy tortoise :(

phoe commented 2 years ago

Do you need any support with that - as in, what would happen to me if I did things the naïve way and made a quicklisp-projects issue pointing to the current master branch?

Zulu-Inuoe commented 2 years ago

I honestly don't remember the status of what things are WIP. I believe there's still a couple of issues with stringify, including the floating point printer, the string replacer, and the auto-coercion stuff in general. As before it's ultimately: I still don't know if it offers anything truly above and beyond the other json libs on QL already.

The other issue is that if we make available a beta version as-is, it'll be hard to fix things without breaking users - the auto-coercion stuff especially scares me.

Zulu-Inuoe commented 1 year ago

Alright this is actually finally happening. All my remaining stuff for v1 are resolved and I've decided what to leave out for a v2.

Code-wise, the true final final stuff is whether or not I keep the alist/plist heuristics in write-value which I am awaiting review. The other changes are things I can do without breaking backwards-compat, like updating docs and optimizing my string-on-string parser

Zulu-Inuoe commented 1 year ago

https://github.com/quicklisp/quicklisp-projects/issues/2256

fukamachi commented 1 year ago

Great, thank you!

phoe commented 1 year ago

Added and available; huge thanks! I think this issue can now be closed.

Zulu-Inuoe commented 1 year ago

Thanks for the heads up, @phoe