Best way to approve images in a PR

Hey all,

I have a runner on a merge request thats checks all the screens of my application and takes screenshots of each screen and compares it to the previously taken one. Long story short every screen that has been changed since the last merge will be written into a file. Now I want however reviews the code to also review the screenshots. What would be a good way to do this? I was thinking maybe adding the screenshots to the merge request description. But would love to hear your take on this.

Many thanks

1 Like

Hello @Cedrick! Welcome to the forum :wave:

Adding those images as comments to the MR is probably the best way - that way each one can quickly and easily become a discussion. Using the API to do that is probably your best bet!

3 Likes

Thanks so much, starting a thread on the MR indeed sounds like a good way of solving this! Will implement this as a solution, thanks again!

2 Likes