Use of send.bind(send, 'METHOD') prevents tree shaking with Rollup

I’m working with the next v2 branch! This is so minor because it’s like… four lines. 😅

I’m working on a library that bundles in httpie instead of importing it (because it’s so small, so why not? 🙌 ) and I noticed that even though I was only importing post the other four helpers were also there. I guess Rollup is being extra careful with bind and includes them in case of any side effects?

When I converted it to something like this (not necessarily recommending it):

export const get = (...args) => send('GET', ...args);
export const post = (...args) => send('POST', ...args);
export const patch = (...args) => send('PATCH', ...args);
export const del = (...args) => send('DELETE', ...args);
export const put = (...args) => send('PUT', ...args);

Then it’d tree shake just fine.

[lukeed]

Whichever you prefer/are using. I’ll comb thru Monday to fill in anything missing and publish a patch

[lukeed]

I think I’m going to leave 1.x branch alone. I should get my act together and promote httpie@next to stable before year end. If you think it’s necessary before I get to that, please leave a shout 😃

Closed by your #35 & released in v2.0.0-next.12 – thanks!