Skip to content

Exclude binary files from ai_reviewable_files and include deleted_files

Sincheol (David) Kimrequested to merge
542917-only-include-text-changes-for-duo-code-review into master

What does this MR do and why?

Exclude binary files from ai_reviewable_files

We were seeing text Endoding exceptions caused by the images included in the merge request. We should exclude them as Duo Code Review is text based review only currently.

Also, include deleted_files as reviewable file as there is no reason we should exclude it.

References

#542917 (closed)

Screenshots or screen recordings

Before After

How to set up and validate locally

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #542917 (closed)

Merge request reports