search

ottypes / json0

Version 0 of the JSON OT type
447 stars 64 forks source link

Document subtypes and deprecate old text ops #2

Closed devongovett closed 9 years ago

devongovett commented 10 years ago

We've supported subtypes since https://github.com/ottypes/docs/pull/9. This PR updates the readme with documentation for them, and deprecates the old text ops, which can now be represented as subtype ops. IMO we should eventually completely remove the old string ops since they are internally converted to and from subtype ops anyway, but that would break backward compatibility.

btw, I'd have made a branch on the main project instead of a fork, but the "Collaborators" team on Github doesn't have write access to this (or any) repository. :smile:

josephg commented 9 years ago

Sorry I've been off the radar for the last couple of months - I've been on holiday from programming.

Thanks for your work but I actually don't think its a good idea to deprecate the sd: and si: ops. I want the spec of JSON0 to stay static (basically) forever, but going forward replace it with the new JSON type (which isn't ready yet.)

I'm adding collaborators to this repo (oops). You're welcome to merge most of the changes from this PR, but please leave the documentation for sd: and si: as-is (or mention that internally its just rewritten using subtypes anyway).

devongovett commented 9 years ago

Updated and merged. Sorry it took so long.