Skip to content

Truncate commits that exceeds the new commits note display limit

What does this MR do and why?

This MR truncates new commit summary if there are more than 10 commits. This is to avoid performance hits caused by pushes that includes a large number of commits. It provides a summary line for them instead.

NOTE: .. notation doesn't seem to work for some reason and that's why I went with using the parent commit sha, but I feel I should include .. there as a backup to match the existing logic for generating existing commits summary line. I'll create an issue for that after this as that's a separate issue.

Related to: #368972 (closed)

Screenshots or screen recordings

With commits less than 10 (same as before)

Screenshot_2023-02-15_at_3.20.02_pm

With commits more than 10

Screenshot_2023-02-15_at_3.17.16_pm

With additional rebased commits

Screenshot_2023-02-15_at_3.00.23_pm

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Related to #368972 (closed)

Edited by Sincheol (David) Kim

Merge request reports