Skip to content

Conversation

@toshiya
Copy link

@toshiya toshiya commented May 28, 2016

Fix for the CI problmes in #9

toshiya added 2 commits May 24, 2016 23:35
4.1.  add
When the operation is applied, the target location MUST reference

4.2. remove
The target location MUST exist for the operation to be successful.
1. fix for the parallel execution in the latest test-unit.
2. specify rake version in Gemfile for ruby 1.9.2 because the latest
rake requires > ruby 1.9.3.
@coveralls
Copy link

coveralls commented May 28, 2016

Coverage Status

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

@toshiya
Copy link
Author

toshiya commented May 28, 2016

@guillec

Now CI passed. Thank you for your reviews.

@toshiya
Copy link
Author

toshiya commented Jun 4, 2016

@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

@guillec
Copy link
Owner

guillec commented Jun 6, 2016

@toshiya will look at this this week. thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants