JosiahWI commented on PR #11945: URL: https://github.com/apache/trafficserver/pull/11945#issuecomment-2910724812
I have not removed the dependency of `Directory` on `StripeSM` or `Stripe`, but after looking at this multiple times, I'm not sure it's a good idea to try to force that. It would be nice to have a cleaner dependency structure, but we should aim to simplify things, not add complexity. This PR cleans up some call sites and makes it clearer what data is required to carry out each operation. Therefore, I think it is a good change as-is. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
