support PUT with empty body parameter#322
Conversation
|
NetflixOSS » feign » feign-pull-requests #201 SUCCESS |
There was a problem hiding this comment.
8.14 has left the building! 8.15 :)
|
PUT with no content is semantically weird, but yeah understood. LGTM after filing into 8.15 in the changelog |
|
NetflixOSS » feign » feign-pull-requests #202 SUCCESS |
|
@pnepywoda mind squashing into one commit? easier for me to click merge vs cherry-pick and squash on my end |
follow-up commit to OpenFeign#271
|
NetflixOSS » feign » feign-pull-requests #203 SUCCESS |
support PUT with empty body parameter
|
thx! let me know if a release is urgent |
|
We are waiting on this to get released...if it could happen this week it On Tue, Jan 26, 2016 at 2:15 AM, Adrian Cole notifications@github.com
|
|
@adriancole actually it's become urgent on our end. Any possibility of release soon? |
|
Yep. Im in transit.
Basically this needs to be cherry picked into 8.x branch and a tag pushed.
I'm embarrassingly without a backup else I'd ask someone else to do this.
@spencergibb I doubt you have access, but would you be a backup for feign?
If so, we can get wheels turning on that.
|
|
I'd be happy to be the backup, though I haven't returned from surgery leave yet. |
|
haha ok so you might be a poor choice of backup at the moment :) Hope you
well on recovery!
|
|
8.14.2 on the way
|
support PUT with empty body parameter
4 participants
follow-up commit to #271