bdelacretaz commented on pull request #18: URL: https://github.com/apache/sling-org-apache-sling-jcr-repoinit/pull/18#issuecomment-857530316
Looks good to me, just one nitpick on test coverage. Building with `-P jacoco-report` shows a few methods in `RetryableOperationResult` which are not covered by tests: `shouldRetry`, `getFailureTrace` and `withJitterMsec`. If you can cover those as well that would be fantastic. And you can then merge, from my point of view. Thank you for this! -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
