-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update aws cli path #8
Conversation
This pull request introduces 2 alerts when merging 69c5d83 into ab17048 - view on LGTM.com new alerts:
|
This pull request introduces 3 alerts and fixes 1 when merging 942b1ea into ab17048 - view on LGTM.com new alerts:
fixed alerts:
|
This pull request introduces 2 alerts when merging 353746d into ab17048 - view on LGTM.com new alerts:
|
This pull request introduces 2 alerts when merging 4a026d4 into ab17048 - view on LGTM.com new alerts:
|
thank you for the PR @ZsoltPath however I don't think this will work. on the normal runners everything appears to be working (see workflows for integration tests), if this fails on a self-hosted runner then I will need to check that out to see whats going on. closing this out for now |
@bryantbiggs I am definitely running this on public runners and need that path change. |
Description
The actual executable in the unzipped package is under aws/dist directory
Motivation and Context
#7