Updated boto3, removed deprecated import path#60
Conversation
|
sigh. More investigation into why tests are failing needed I guess. |
|
@LindsayHill Just a quick question, how feasible to have boto3 pack with only just two actions which essential need to perform all aws activities? Those two actions are;
So that users can have the choice to use either regular aws pack or this boto3 pack. |
|
@Susanthab probably better to ask broader questions via Slack or a new GH issue, rather than on this PR. You probably want to have a read through some of the other discussions around the AWS pack, and talk to people like @AndyMoore111 who are planning a new way of doing things. Possible starting point is here StackStorm-Exchange/exchange-incubator#10, and the issues/PRs linked from there. e.g. "Discussion on the future of the AWS pack" |
|
thanks. will do.
Get Outlook for Android<https://aka.ms/ghei36>
…________________________________
From: Lindsay Hill <notifications@github.com>
Sent: Monday, December 18, 2017 12:33:08 PM
To: StackStorm-Exchange/stackstorm-aws
Cc: Susantha Bathige; Mention
Subject: Re: [StackStorm-Exchange/stackstorm-aws] Updated boto3, removed deprecated import path (#60)
@Susanthab<https://github.com/susanthab> probably better to ask broader questions via Slack or a new GH issue, rather than on this PR. You probably want to have a read through some of the other discussions around the AWS pack, and talk to people like @AndyMoore111<https://github.com/andymoore111> who are planning a new way of doing things.
Possible starting point is here StackStorm-Exchange/exchange-incubator#10<StackStorm-Exchange/exchange-incubator#10>, and the issues/PRs linked from there. e.g. "Discussion on the future of the AWS pack<#55>"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#60 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/APOtzRiVsz5u2MGtdTlUfLJUfPhlirppks5tBr3zgaJpZM4RBc9s>.
|
Also cleaned up mixed up CHANGES.md vs CHANGES.rst that still existed on this branch.
This replaces #59