[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7304: Added the separated workflow badges to readme

[via GitHub]

ocket8888 commented on code in PR #7304:
URL: https://github.com/apache/trafficcontrol/pull/7304#discussion_r1082955791


##########
README.md:
##########
@@ -32,6 +32,7 @@ Apache Traffic Control allows you to build a large scale 
content delivery networ
 [![Youtube 
Subscribe](https://img.shields.io/youtube/channel/subscribers/UC2zEj6sERinzx8w8uvyRBYg?style=social&label=Apache%20Traffic%20Control)](https://www.youtube.com/channel/UC2zEj6sERinzx8w8uvyRBYg)
 
 [![GitHub release (latest 
SemVer)](https://img.shields.io/github/v/release/apache/trafficcontrol)](https://github.com/apache/trafficcontrol/releases)
+![Github commits since 
release](https://img.shields.io/github/commits-since/apache/trafficcontrol/RELEASE-7.0.1/master)

Review Comment:
   You should probably use the latest release, since I think that's the 
intention and then it won't need to be updated when we make new releases. I.E.
   
   ```markdown
   ![Github commits since 
release](https://img.shields.io/github/commits-since/apache/trafficcontrol/latest/master)
   ````
   
   rendered:
   
   ![Github commits since 
release](https://img.shields.io/github/commits-since/apache/trafficcontrol/latest/master)



##########
README.md:
##########
@@ -52,18 +53,17 @@ __Code Status__ [^1]
 
 __Test Status__ [^1]
 
-[![Go Unit 
Tests](https://github.com/apache/trafficcontrol/workflows/Go%20Unit%20Tests/badge.svg)](https://github.com/apache/trafficcontrol/actions/workflows/go.unit.tests.yaml?query=branch%3Amaster)
 
[![Codecov](https://codecov.io/gh/apache/trafficcontrol/branch/master/graph/badge.svg)](https://app.codecov.io/github/apache/trafficcontrol/tree/master)
-
-[![TO Go Client Integration 
Tests](https://github.com/apache/trafficcontrol/workflows/TO%20Go%20Client%20Integration%20Tests/badge.svg)](https://github.com/apache/trafficcontrol/actions/workflows/traffic-ops.yml?query=branch%3Amaster)
 
[![Codecov](https://codecov.io/gh/apache/trafficcontrol/branch/master/graph/badge.svg?flag=traffic_ops_integration)](https://app.codecov.io/github/apache/trafficcontrol/tree/master/traffic_ops)
-
-[![Traffic Router 
Tests](https://github.com/apache/trafficcontrol/workflows/Traffic%20Router%20Tests/badge.svg)](https://github.com/apache/trafficcontrol/actions/workflows/tr.tests.yaml?query=branch%3Amaster)
 
[![Codecov](https://codecov.io/gh/apache/trafficcontrol/branch/master/graph/badge.svg?flag=traffic_router_unit)](https://app.codecov.io/github/apache/trafficcontrol/tree/master/traffic_router)
-
-[![TP Integration 
Tests](https://github.com/apache/trafficcontrol/workflows/TP%20Integration%20Tests/badge.svg)](https://github.com/apache/trafficcontrol/actions/workflows/tp.integration.tests.yml?query=branch%3Amaster)
 
-[![TM Integration 
Tests](https://github.com/apache/trafficcontrol/workflows/TM%20Integration%20Tests/badge.svg)](https://github.com/apache/trafficcontrol/actions/workflows/tm.integration.tests.yml?query=branch%3Amaster)
-[![TR Ultimate Test 
Harness](https://github.com/apache/trafficcontrol/workflows/TR%20Ultimate%20Test%20Harness/badge.svg)](https://github.com/apache/trafficcontrol/actions/workflows/tr-ultimate-test-harness.yml?query=branch%3Amaster)
-[![T3C Integration 
Tests](https://github.com/apache/trafficcontrol/workflows/T3C%20Integration%20Tests/badge.svg)](https://github.com/apache/trafficcontrol/actions/workflows/cache-config-tests.yml?query=branch%3Amaster)
-[![TC Health Client Integration 
Tests](https://github.com/apache/trafficcontrol/workflows/TC%20Health%20Client%20Integration%20Tests/badge.svg)](https://github.com/apache/trafficcontrol/actions/workflows/health-client-tests.yml?query=branch%3Amaster)
-
+| Unit Tests                                                                   
                                                                                
                                                                                
                                                                                
                                                                                
                                                    | Integration Tests         
                                                                                
                                                                                
                                                                                
                                                                                
                                                                                
   | 

Review Comment:
   Do you think this table should have a "Component" column? So I can easily 
understand why two badges are on the same line, and what the dashes represent.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org