swatimodi-scout
swatimodi-scout
> The git history of this PR seems odd, lots of those commits are unrelated. Do you mind cleaning it up so it only contains commits related to this change?...
@mikeee @javier-aliaga Requested changes are done. Please review it.
@mikeee @acroca Is there anything required from our side now?
@mikeee @acroca Could you please merge this PR if everything looks good to you?
> The changes to `/common/authentication/aws` should also be reverted if the package is not being referenced in this PR. Are you talking about this change? - The changes to /common/authentication/aws...
> My last review comment still stands Done please review the changes
> Lgtm, just noting that I haven't got an environment set up to test this and we don't have certification/conformance test coverage for this binding. A bonus definitely would be...
@javier-aliaga Can we now merge this PR?
> LGTM. Still this will not be available until 1.17 (planned for January) Got it, thanks for the update.