Return ES2015 Proxy object #29

@schnittstabil Interested in exploring this?

I'm requiring Node.js 4 in the next version, so we can safely use Proxy then.

At least v4.5.0 (LTS) and v5.12.0 don't support Proxies :cry:

Oh no. I was sure Proxy was in Node.js 4 for some reason...

@sindresorhus If you'd like to give me access rights, I'll create a proxy-branch and push some v6 code for further usage…

Done :)

Yes, good idea. Can at least explore it now. There might be some performance issues using Proxies.

There might be some performance issues using Proxies.

I've experienced the same issue - solved it by introducing a cache: https://github.com/sindresorhus/pify/blob/proxy/index.js#L44 :bowtie:

@schnittstabil Nice! Already looks very good. I commented some quick feedback on the commit, but haven't tried it out yet. When that's resolved, can you open a PR? Would make it easier to review.

Yes but why? The code doesn't seem much/any shorter/simpler/faster than the regular version.

@bfred-it Some things will just work compared to now, like ES6 classes, this binding, etc.

Anyone wants to step in and finish https://github.com/sindresorhus/pify/pull/32? Including handle my review comments. I'll put a bounty on this shortly. I think it's better now to just have the proxy interface instead of both. Proxy is pretty optimized in newer Node.js versions. This task includes adding tests to handle all the things mentioned in the open issues too.

@issuehunt has funded $70.00 to this issue.

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

I'll take it.

@sindresorhus has rewarded $63.00 to @frangio. See it on IssueHunt

• :moneybag: Total deposit: $70.00
• :tada: Repository reward(0%): $0.00
• :wrench: Service fee(10%): $7.00