garydgregory commented on PR #328: URL: https://github.com/apache/commons-net/pull/328#issuecomment-2652029212
I'm not a fan of a PR for a one-off style changes to a single method. It feels like change for the sake of change. We don't use `CsvSource` at all ATM. I'm not opposed to using it but if used, it should be done consistently in the whole class, maybe the whole package, maybe in the whole component. It's a discussion worth having. What do others think? -- 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]
