search

timoschwarzer / gitlab-monitor

A web-based monitor dashboard for GitLab CI
https://gitlab-monitor.timoschwarzer.com
MIT License
482 stars 128 forks source link

Feature: show number of merge requests in project card. #95

Open Feng-Yin opened 5 years ago

Feng-Yin commented 5 years ago

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.

timoschwarzer commented 5 years ago

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 commented 5 years ago

I'm open to more opinions though.

Feng-Yin commented 5 years ago

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

devpie commented 5 years ago

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.

timoschwarzer commented 5 years ago

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
Morl99 commented 3 years ago

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?

timoschwarzer commented 3 years ago

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