sebbASF opened a new issue #114: URL: https://github.com/apache/incubator-ponymail-foal/issues/114
https://github.com/apache/incubator-ponymail-foal/blob/4def8545bb894f1890f46d5a2628c54cb6e98831/tools/plugins/textlib.py#L40 It's not ideal for utility methods to use print to signal errors, as the caller has no choice as to whether the message appears or not. Also the function currently can return two different types. Callers need to check the return type on every call. It would be easier for callers if the function raised an error instead. -- 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]
