refactor code

[alphmt]

Please check if the PR fulfills these requirements

• Followed the Contributing guidelines.
• Tests for the changes have been added (for bug fixes/features) with 100% code coverage.
• Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Refactor code to have cacheable.request() instead of cacheable.createCacheableRequest()

[alphmt]

@jaredwray @szmarczak is this now ok?

[codecov]

Codecov Report

Merging #188 (1c0ff13) into main (01224b8) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #188   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines          158       158           
  Branches        39        39           
=========================================
  Hits           158       158           

Impacted Files	Coverage Δ
src/index.ts	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[szmarczak]

Sorry for late response. This is not what I meant. I meant that the cacheableRequest should be cached and exposed as request. Meaning, this should work:

const cacheableRequest = new CacheableRequest(http.request);
const cacheReq = cacheableRequest.request('http://example.com', cb);
cacheReq.on('request', req => req.end());

I'd make request bound to the current instance, so it would be backwards compatible:

const request = new CacheableRequest(http.request).request;
const cacheReq = request('http://example.com', cb);
cacheReq.on('request', req => req.end());

Anyway, it might be too late for this change. I'm ok with the current solution.

[jaredwray]

@szmarczak - sorry for the confusion and I will look at what it would take to do that.