search

crs4 / life_monitor

Workflow Sustainability Service
MIT License
4 stars 4 forks source link

Remove "Suites" column from Dashboard #282

Closed ilveroluca closed 1 year ago

ilveroluca commented 1 year ago

It looks like pretty much all workflows have a single test suite. I propose we remove the "Suites" column from the dashboard since it doesn't seem very useful for most (all?) cases.

What do you think @kikkomep @simleo

simleo commented 1 year ago

I would rather not remove this feature just because no one used it so far. We might need it in the future. It's reasonable to split up tests into multiple suites if you have many. It might also be useful for multi-workflow repositories, depending on how they end up being handled.

ilveroluca commented 1 year ago

I'm not suggesting to remove suites as a concept, but just removing the column from the table.

Il lun 21 nov 2022, 17:40 Simone Leo @.***> ha scritto:

I would rather not remove this feature just because no one used it so far. We might need it in the future. It's reasonable to split up tests into multiple suites if you have many. It might also be useful for multi-workflow repositories, depending on how they end up being handled.

— Reply to this email directly, view it on GitHub https://github.com/crs4/life_monitor/issues/282#issuecomment-1322346293, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALO5WG2DLYTFOXZAR6JRZ3WJOQYDANCNFSM6AAAAAASGXCQCY . You are receiving this because you authored the thread.Message ID: @.***>

simleo commented 1 year ago

The main advantage would be helping visualization in small screens. Otherwise, I don't think it hurts that much.