Feature: Expose updateMergeRequestInfo for gitlab api#1391
Feature: Expose updateMergeRequestInfo for gitlab api#1391glensc wants to merge 2 commits intodanger:mainfrom
Conversation
|
The CI error is not from changes of this PR: |
|
actually, I ran such code in my danger plugin: await gitlab.api.MergeRequests.edit(mr.project_id, mr.iid, { assigneeId: mr.author.id })and I'm getting 403, what's with that? EDIT: I think my token has no write accesss |
| fileContents: api.getFileContents, | ||
| addLabels: api.addLabels, | ||
| removeLabels: api.removeLabels, | ||
| updateMergeRequestInfo: api.updateMergeRequestInfo, |
There was a problem hiding this comment.
Should the method be added to utils or top level (aside api: xxx)?
|
I don't think we should do this, as this fn is just an alias to |
|
The wrapped methods add debug calls. it would be useful to see debug logs when troubleshooting. But this all started because incompatible api change: |
|
I'd imagine there would be similar debugging env vars on the gitlab API instance too - yeah, |
Similarly to #1353, expose
updateMergeRequestInfomethod to be able to edit merge requests from danger code.