Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

aws/session: Fix formatting bug in doc. #2294

Merged
merged 1 commit into from
Dec 4, 2018
Merged

Conversation

a4abhishek
Copy link
Contributor

@a4abhishek a4abhishek commented Nov 21, 2018

This PR fixes a minor issue in aws/session/doc.go where the developer mistakenly used format specifiers in logger.Println.

Changes committed:
modified: aws/session/doc.go

Signed-off-by: Abhishek Kashyap avskksyp@gmail.com

This commit fixes a minor issue in `aws/session/doc.go` where the
developer mistakenly used format specifiers in `logger.Println`.

Changes committed:
	modified:   aws/session/doc.go

Signed-off-by: Abhishek Kashyap <avskksyp@gmail.com>
@a4abhishek
Copy link
Contributor Author

a4abhishek commented Nov 24, 2018

Hey @jasdel , What is it which is keeping us from merging this PR?

@jasdel
Copy link
Contributor

jasdel commented Nov 24, 2018

Hi @a4abhishek thanks for creating the fix for this documentation. The change is good and we'll be able to merge it. We're holding off merging non-critical updates during re:Invent. We'll be be able to merge in this change after re:Invent at the latest.

@a4abhishek
Copy link
Contributor Author

Sure @jasdel , thanks.

@jasdel jasdel changed the title This PR fixes a minor issue in doc of aws/session aws/session: Fix formatting bug in doc. Dec 4, 2018
@jasdel jasdel merged commit fb43c28 into aws:master Dec 4, 2018
@a4abhishek a4abhishek deleted the fix-doc branch December 5, 2018 10:33
@aws-sdk-go-automation aws-sdk-go-automation mentioned this pull request Dec 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants