Fix comment API paths

[ethantkoenig]

Corrects the following comment API routes

• PATCH /:owner/:repo/issues/:index/comments/:id -> PATCH /:owner/:repo/issues/comments/:id
• DELETE /:owner/:repo/issues/:index/comments/:id -> DELETE /:owner/:repo/issues/comments/:id

The previous routes ignored the :index parameter, and were in conflict with the Github API. The previous route are marked as deprecated, but have not been removed.

[lafriks]

Can we create new v3 API route group for all these breaking changes?

[ethantkoenig]

@lafriks IMO, that seems like overkill. I doubt that fixing a few broken routes will cause much disruption.

[lafriks]

If we are sure no tool is using it I'm ok with that

[ethantkoenig]

@lafriks I'm not aware of any such tool, but of course I can't guarantee that no such tool exists.

[codecov-io]

Codecov Report

Merging #2813 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2813   +/-   ##
=======================================
  Coverage   27.24%   27.24%           
=======================================
  Files          89       89           
  Lines       17640    17640           
=======================================
  Hits         4806     4806           
  Misses      12146    12146           
  Partials      688      688

---

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 876bafb...492ff2f. Read the comment docs.

[bkcsoft]

Well, any way to know if it'll break things is to do it and see if anyone complains

LGTM

[ethantkoenig]

@lafriks Instead of removing the old, incorrect routes, I've marked them as deprecated. This PR is no longer breaking.

[lafriks]

LGTM