#993 Gitlab diff api seems to mismatch param

[DoubleDPro]

For actualization MergeRequestApi:

• added methods "getMergeRequestDiffs" for GitLab Endpoint: GET /projects/:id/merge_requests/:merge_request_iid/diffs
• renamed method "getMergeRequestDiffs" for GitLab Endpoint: GET /projects/:id/merge_requests/:merge_request_iid/versions

Fixes #993

[jmini]

Thank you for this PR 👍
I will have a look next week.

[jmini]

The issue I see with the suggested change, is that this is breaking, because getMergeRequestDiffs() is changing.

For GET /projects/:id/merge_requests/:merge_request_iid/commits the methods is just named getCommits()

I suggest an other naming:

• Instead of getMergedRequestXxxx(..) just use getXxxx(..) (remove the MergedRequest part)
• for GET /projects/:id/merge_requests/:merge_request_iid/diffs
  • --> getDiffs(..)
• for GET /projects/:id/merge_requests/:merge_request_iid/versions
  • --> getVersions(..)

So getMergeRequestDiffs() is deprecated and recommend to use getVersions(..) instead.

I also created the MergeRequestDiffVersion as suggested in #993 (comment) which is a subset of MergeRequestDiff.
So the MergeRequestDiff is extending from MergeRequestDiffVersion.

When a single version is loaded the complete object is returned: MergeRequestDiff.
When multiple versions are loaded only a list of MergeRequestDiffVersion is returned.

[jmini]

@DoubleDPro I have pushed a new commit to your branch. Let me know what you think.

[DoubleDPro]

@jmini LGFM! Let's merge.

[jmini]

Merged to main and then to 6.x where breaking changes are allowed. So on the 6.x branch I removed the deprecated methods.