-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Drop deprecated JSON.mapping #9527
Conversation
The Should I create a |
I think |
I'd suggest scoping that to |
Changing the name of the exception would introduce a breaking change. Looking at YAML, there's just |
Nah, after a few months after 1.0.0 nobody will care about that small breaking change. Plus the 1.0.0 release will break all shards so they will have to upgrade anyway. |
Names are not that tidy. The following exists I would prefer to avoid using |
Use github:crystal-lang/json_mapping.cr
3b8a1e2
to
14bea63
Compare
…rystal-lang/crystal#9527); (Refactor to use JSON::Serializable instead?)
* drhuffman12/upgrade_to_crystal_1.1.1 "json_mapping" is now a shard (crystal-lang/crystal#9527); (Refactor to use JSON::Serializable instead?) * drhuffman12/upgrade_to_crystal_1.1.1 avoid "Missing ENV key" errors; add more db conn env var's * drhuffman12/upgrade_to_crystal_1.1.1 TODO: Why "CouchDB::Response::Vendor#version" not getting parsed in? * drhuffman12/upgrade_to_crystal_1.1.1 Allow CouchDB '3.x.x' (not just '2.x.x'). (TODO: Add applicable tests for '3.x.x' changes.); TODO: Why "CouchDB::Response::Vendor#version" not getting parsed in? * drhuffman12/upgrade_to_crystal_1.1.1 Bump shard version to "0.4.0"; comment out Crystal version requirement (note compatibility elsewhere?) * drhuffman12/upgrade_to_crystal_1.1.1 cleanup re `Vendor#version` * drhuffman12/upgrade_to_crystal_1.1.1 Add myself as a contributor.
Use github:crystal-lang/json_mapping.cr as a drop-in replacement.