net/url: Adds patch-pure-port and patch-impure-port #2464
net/url: Adds patch-pure-port and patch-impure-port #2464DarrenN wants to merge 3 commits intoracket:masterfrom
Conversation
Adds the HTTP verb PATCH (https://en.wikipedia.org/wiki/Patch_verb) to net/url for completeness.
|
Hmmm the two fails seem to be from an unrelated package? https://travis-ci.org/racket/racket/jobs/491101788#L2808 |
|
Hi and thanks for the patch. As @greghendershott mentioned on Slack, |
|
@jeapostrophe do you think there would be value in marking this library as deprecated in the documentation? |
|
Sure |
|
In my experience with Racket, it's very rare for things to be marked as "deprecated" in the sense of "We want to remove X and break working applications, because reasons. We hope doing something rude will seem less rude if we do it slowly." 😄 So I think the most helpful thing would be a At least, that's my $0.02, but I'm not on the hook to maintain |
|
I was thinking of a note like the one for net/cookies |
Background
net/urlwas missing functions to make HTTP requests with thePATCHverb. API endpoints acceptingPATCHare becoming more common.This PR adds
patch-pure-portandpatch-impure-port.