chore: upgrade koa-convert dependency to ^2.0.0

[atombrenner]

fix issue #1534

[codecov]

Codecov Report

Merging #1535 (904f8ab) into master (eb51cf5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #1535   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         4           
  Lines          489       489           
  Branches       136       136           
=========================================
  Hits           489       489           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eb51cf5...904f8ab. Read the comment docs.

[miwnwski]

I agree it should be upgraded (and removed in next) but this would result in a breaking change (I don't really remember why convert got bumped major, maybe you can enlighten me if you recently checked it out?).

This will have to wait to next version of Koa, though any possible next Koa dependency would actually remove this dependency.

[atombrenner]

(I don't really remember why convert got bumped major, maybe you can enlighten me if you recently checked it out?).

As far as I can see, the change is the dependency upgrade of koa-compose@3.2.1 to koa-compose@4.1.0 plus some more cosmetic refactorings.

[miwnwski]

Well in that case, it LGTM!

[miwnwski]

Holding off merge a little to give others some time to chime in

[3imed-jaberi]

lgtm +1

whisper: we must move quickly to work on next release of Koa. #1508