-
Notifications
You must be signed in to change notification settings - Fork 191
Integrate crate-metadata fork #58
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
Comments
I merged the json-schema ones. Happy to merge a PR that copies them over. |
The only wrinkle is I'm not sure if we diverged from the spec at all with those patches. |
hmm...how can you tell? |
I probably just have to look through. Was there any particular patchset you wanted pulled forward? |
hmm...at the moment we are dependent on some of the introduced operations, though whether we actually use them in practice, i'm not sure. |
after some tidy up i think we can return to using this library without the crate-metadata changes - now that this is being maintained i suspect it's better to stay closer to the RFC standard without the extra operations. were there any bug-fixes or improvements on the fork that aren't related to the extra operations? |
Are there any plans to integrate the changes in https://github.com/crate-metadata/json-patch?
The text was updated successfully, but these errors were encountered: