Style improvements

[brianmhunt]

Much like #10, consider some ES6ish things:

1. The defineProperty code is unnecessarily verbose. Consider
   i. changing the defineProperty to getters e.g. get json () { return this.response.then(resp => resp.clone().json()) } or
   ii. using ['json', 'blob', ...].forEach((n) => Object.defineProperty(this, n, {get: () => this.response.then(r => r.clone()[n]()}) or
   iii. using defineProperties
2. Define setHeaders(obj = {}) and remove if (opts.headers)
3. change makeBody to a named function so that its name shows up when debugging
4. change setHeader (key, value) { this.setHeaders({ [key]: value }) }
5. have _args return this, then change all the http method methods (get, post, ...) to return this._args(...args)

[gr2m]

done via #38

• The defineProperty code is unnecessarily verbose. Consider
  i. changing the defineProperty to getters e.g. get json () { return this.response.then(resp => resp.clone().json()) } or
  ii. using ['json', 'blob', ...].forEach((n) => Object.defineProperty(this, n, {get: () => this.response.then(r => r.clone()[n]()}) or
  iii. using defineProperties

PR at #56

• change makeBody to a named function so that its name shows up when debugging
• change setHeader (key, value) { this.setHeaders({ [key]: value }) }
• Define setHeaders(obj = {}) and remove if (opts.headers)

I think that’s obsolete

• have _args return this, then change all the http method methods (get, post, ...) to return this._args(...args)

[brianmhunt]

I find this oddly satisfying. 😉