ocket8888 opened a new pull request #4948:
URL: https://github.com/apache/trafficcontrol/pull/4948


   ## What does this PR (Pull Request) do?
   - [x] This PR not related to any Issue
   
   This PR will run Go unit tests on all PRs that modify go code, as well as 
all pushes to any branch that modifies go code. Specifically, it checks 
modification in paths that match:
   ```gitignore
   grove/**.go
   lib/**.go
   traffic_monitor/**.go
   traffic_ops/traffic_ops_golang/**.go
   traffic_ops_ort/atstccfg/**.go
   ```
   and runs the unit tests in those same paths when modifications are found.
   
   I know that this sort of got shut down on the mailing list, but [that 
discussion](https://lists.apache.org/thread.html/ec503661975dc9eb0c3719bcaab3b758b5b7489ebec93f95f7f9416b%40%3Cdev.trafficcontrol.apache.org%3E)
  was about a year ago now, and we still don't have unit tests running on our 
PRs. So this PR adds them. If we want to merge it.
   
   ## Which Traffic Control components are affected by this PR?
   none
   
   ## What is the best way to verify this PR?
   Look at the "Checks" tab.
   
   ## The following criteria are ALL met by this PR
   - [x] This PR includes tests
   - [x] Documentation is unnecessary
   - [x] An update to CHANGELOG.md is not necessary
   - [x] This PR includes any and all required license headers
   - [x] This PR does not include a database migration
   - [x] This PR **DOES NOT FIX A SERIOUS SECURITY VULNERABILITY** (see [the 
Apache Software Foundation's security 
guidelines](https://www.apache.org/security/) for details)
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to