cammckenzie commented on pull request #393: URL: https://github.com/apache/curator/pull/393#issuecomment-892230176
Having some issues putting comments against the file. The main fix looks good to me. As for tests, it looks like this class has none currently, any additions would be welcome though. I'm not sure why you need to make fixAclList static though? It can certainly be static, but it's only used by this class, so it does't need to be. -- 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]
