gitlab-monitor icon indicating copy to clipboard operation
gitlab-monitor copied to clipboard

Feature: show number of merge requests in project card.

Open Feng-Yin opened this issue 5 years ago • 7 comments

Hi, awesome job on the dashboard! Pls consider adding merge request related info to the dashboard. We had the issue of promoting the actively review the merge request in a timely manner.

Feng-Yin avatar Sep 06 '19 07:09 Feng-Yin

Hmm, gitlab-monitor's original intention is to display real-time pipeline information, which merge requests are not part of. So I don't think I'll implement that really specific feature.

timoschwarzer avatar Sep 06 '19 07:09 timoschwarzer

I'm open to more opinions though.

timoschwarzer avatar Sep 06 '19 07:09 timoschwarzer

I got a patch for showing open merge request on project card top-right corner showOpenMergeRequestCount.patch.txt

Feng-Yin avatar Sep 16 '19 13:09 Feng-Yin

I think showing this kind of information is a great thing. At my last employer we build our own Web-Application for additionally showing open MRs respectively open code reviews, but in an additional badge, which went from green, when no MRs where available to orange to red, when there were more than x and y MRs. This showed us to keep up with the stuff from others and review their code, which really enhanced i.e. our lead time.

devpie avatar Oct 18 '19 15:10 devpie

Alright, as there seems to be demand, I'd be fine with little badges at the top right corner of each project card. Users can enable the badges with a config in projectConfig.

e.g.

projectConfig:
  '*': # or 'johndoe/someproject'
    badges:
      mergeRequests: true

timoschwarzer avatar Oct 18 '19 16:10 timoschwarzer

I think that would be helpful. Do you still have plans to implement this? Or does 'help-wanted' mean you would be open to PRs about this?

Morl99 avatar Dec 02 '20 14:12 Morl99

@Morl99 probably not going to implement this myself, but I'm open for PRs :)

timoschwarzer avatar Dec 02 '20 19:12 timoschwarzer