You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@trafficcontrol.apache.org by "ericholguin (via GitHub)" <gi...@apache.org> on 2023/02/01 15:30:11 UTC

[GitHub] [trafficcontrol] ericholguin commented on pull request #7287: Fix t3c test duplication and upgrade to v4 (fix conflicts)

ericholguin commented on PR #7287:
URL: https://github.com/apache/trafficcontrol/pull/7287#issuecomment-1412254327

   > #7287 changes (or removes) 130 files, which is too large of an impact for a single PR. Would you please break #7287 up into some smaller PRs, each of which could hopefully be reviewed by a single person?
   
   I'm not sure that's necessary. The changes are cookie cutter changes, moving test creation/deletion functions to a shared folder used by both the TO integration and t3c integration tests, it touches a lot of files but these aren't complicated changes.
   


-- 
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-unsubscribe@trafficcontrol.apache.org

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