json-patch icon indicating copy to clipboard operation
json-patch copied to clipboard
verified publisher icon guillec

Fix/add object error handling

Open toshiya opened this issue 9 years ago • 4 comments

Fix for the CI problmes in https://github.com/guillec/json-patch/pull/9

toshiya avatar May 28 '16 08:05 toshiya

Coverage Status

Coverage remained the same at 100.0% when pulling 010e085e2690fd0131ae8ddc75dece81a4ecb592 on toshiya:fix/add_object_error_handling into d3a33b00d49cc2b62864295085f7c6fc1215fe1f on guillec:master.

coveralls avatar May 28 '16 08:05 coveralls

@guillec

Now CI passed. Thank you for your reviews.

toshiya avatar May 28 '16 08:05 toshiya

@guillec

How about this PR?

This PR contains the fix for the following error cases specified in the RFC.

4.1. add When the operation is applied, the target location MUST reference https://tools.ietf.org/html/rfc6902#section-4.1

4.2. remove The target location MUST exist for the operation to be successful. https://tools.ietf.org/html/rfc6902#section-4.2

toshiya avatar Jun 04 '16 13:06 toshiya

@toshiya will look at this this week. thanks!

guillec avatar Jun 06 '16 13:06 guillec