FYI, sometimes its better to create pull requests and ask the community to review the code before the first release.
- there are bad practices like storing non-transient Build classes in Reports. It may cause many issues n the cause of the improper usage.
- FileNotFound exceptions should be converted to AbortException, because you know the failure reason
- Strange JSON handling magic in the publisher code, could be significantly refactored and optimized
- Inconsistent null/empty string checks in parameters from config.xml
- etc.
четверг, 13 августа 2015 г., 11:24:24 UTC+3 пользователь Priti Changlani написал: