rawlinp commented on pull request #5512: URL: https://github.com/apache/trafficcontrol/pull/5512#issuecomment-777635871
I would probably hold off on this one until https://github.com/apache/trafficcontrol/pull/5430 is completed/merged. That PR is actually needed for 5.1. Also, wouldn't this be a good PR to implement the TO Go client best practices discussed on the mailing list? I wasn't sure if we really wanted to make all these breaking signature changes. But if we are, it might also be a good time to move resources into their own "modules" a la `Session.DeliveryServices.Get(...)`, `Session.Servers.Update(...)`? ---------------------------------------------------------------- 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]
