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

Readme: clarify that aws-session-token supported #189

Merged
merged 2 commits into from
Apr 23, 2021
Merged

Readme: clarify that aws-session-token supported #189

merged 2 commits into from
Apr 23, 2021

Conversation

holyjak
Copy link
Contributor

@holyjak holyjak commented Mar 22, 2021

Update README.

I nearly gave up, believing it is not supported, until I read the action.yml. I think it is worth making this clear already in the README.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@allisaurus allisaurus self-assigned this Mar 23, 2021
@allisaurus
Copy link
Contributor

Thanks for contributing @holyjak ! Would you mind changing the commit msg* to be semantic (something like "chore: Add aws-session-token notation to README" so that we can merge this in?

*to clarify, I mean the actual git history/commit msg in your branch, not just the PR title (common point of confusion)

@allisaurus
Copy link
Contributor

@holyjak wanted ping you again in case you were still able + interested in resolving the semantic PR check to get this merged

I nearly gave up, believing it is not supported, until I read the action.yml. I think it is worth making this clear already in the README.
@holyjak
Copy link
Contributor Author

holyjak commented Apr 14, 2021

Fixed. Thank you for the ping, I have not seen your original comment.

@mergify mergify bot merged commit 3042b00 into aws-actions:master Apr 23, 2021
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.

None yet

2 participants