enapps-enorman commented on PR #38: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/38#issuecomment-1270482796
> I wouldn't mind if we keep this functionality, even if it is just a thin wrapper around `Awaitility`. @joerghoh I won't stand in the way of that since this just just for testing, but it seems redundant and the solution is incomplete since it is only handling the 'GET' request scenario. The same problem could arise for a head, post, put, delete, patch or stream action as well. The solution I proposed would handle all other scenarios the same way. Perhaps a simple example in the [readme](https://github.com/apache/sling-org-apache-sling-testing-clients/blob/master/README.md) FAQ would suffice? -- 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]
