dvmarcilio commented on PR #1047: URL: https://github.com/apache/commons-lang/pull/1047#issuecomment-1498740949
Thanks, @laurentschoelens for fixing these and @kinow for pinging me. These 4 methods were among my tool findings. I'm rerunning the tool on the latest commit. I'll be happy to submit more PRs fixing what it finds. Then we can finally close LANG-1681. Should I submit one PR per class or a single (possibly large) PR for all findings? The tool also finds missing `@throws`. I can submit different PRs for those as well. Does that sound good to you, @kinow? -- 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]
