microsoft / PR-Metrics

A GitHub Action & Azure Pipelines task for augmenting pull request titles to let reviewers quickly determine PR size and test coverage.
https://github.com/microsoft/Omex
MIT License
58 stars 12 forks source link

Request: Make PR comment table horizontal vs vertical #399

Closed houserx-jmcc closed 1 year ago

houserx-jmcc commented 1 year ago

Summary

Make the metrics table horizontal vs vertical to save space. This could be an optional format, or just the new default one.

Rationale

When there is a lot of activity in a pull request on GitHub, vertical screen real estate can quickly be eaten up by various Actions and bot comments. By switching the format of the PR metrics table, we can quickly re-gain some of that space.

Additional Information

Current:

Lines
Product Code 0
Test Code 2
Subtotal 2
Ignored Code 23
Total 25

Proposed:

Product Code Test Code Subtotal Ignored Code Total
Lines 0 2 2 23 25
muiriswoulfe commented 1 year ago

Hi @houserx-jmcc,

Thanks for your feedback. I understand the issue you have identified, but unfortunately, this change is out of scope for the project. The reasons are:

I understand that we can provide an option for switching between table formats as you identified, but we don't have the bandwidth for adding and supporting features that we do not envisage being of widespread use. If there is additional demand for this, it's something we can reconsider, but at the moment, we cannot justify this addition.

Thanks once more for your contribution.