Tina Boyce

Results 27 comments of Tina Boyce

Hi @sledigabel, apologies for the delay in getting back to you. I had a discussion with @mapkon and we agreed to get this pull request going. There is currently a...

Closing this issue as a recently merged PR has an update README to cover Ubuntu-specific instructions #1238 . Thank you everyone who has contributed to this discussion.

@fernbach it cannot be merged because there is a branch conflict and missing test coverage. If both conditions are met, then we will be more than happy to merge it.

@mapkon I have added tests which has resulted in a positive coverage https://app.codecov.io/gh/Versent/saml2aws/pull/547 . Is this enough for the PR to proceed?

FYI, updated the test to utilise defer as I forgot it was a thing.

hi @eonpatapon , is this PR still relevant? I've noticed the issue this PR meant to close on merge is already closed.

Ah okay, thanks for checking up on it for me. I'll keep this issue open.

Closing this issue as the current go.mod is using v1.17.0 https://github.com/Versent/saml2aws/blob/f183682460a430642782e958feb8473941a07fbe/go.mod#L25

Thanks for giving us an update @donnyv12 , much appreciated. Was there any resolution to your issue @kun-shi ?

Thanks for your input on this @kieran-lowe . @BamOidaFixMan , is this still an issue?