-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
added my-widget-service for Python #63
Conversation
Update ecs typescript examples to v0.36.1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't looked at the main app body/tested it, just making some general comments to bring the style in line with other examples. If in doubt, have a look at the other examples to get an idea of what needs committing and what doesn't.
python/my-widget-service/my_widget_service/my_widget_service.egg-info/PKG-INFO
Outdated
Show resolved
Hide resolved
python/my-widget-service/my_widget_service/my_widget_service.egg-info/SOURCES.txt
Outdated
Show resolved
Hide resolved
python/my-widget-service/my_widget_service/my_widget_service.egg-info/dependency_links.txt
Outdated
Show resolved
Hide resolved
python/my-widget-service/my_widget_service/my_widget_service.egg-info/requires.txt
Outdated
Show resolved
Hide resolved
python/my-widget-service/my_widget_service/my_widget_service.egg-info/top_level.txt
Outdated
Show resolved
Hide resolved
python/my-widget-service/my_widget_service/my_widget_service_stack.py
Outdated
Show resolved
Hide resolved
python/my-widget-service/my_widget_service/my_widget_service_stack.py
Outdated
Show resolved
Hide resolved
* WIP: Updating Python examples to 0.36.1 * Adding some ECS examples * Adding more ECS examples * Adding final ECS examples
I don't quite understand the build failure. it appears to be npm related and not something I can fix in this commit, is this correct? |
Yeah, I don't think it's related to your changes and I don't think there is anything you can do to fix it in this PR. |
Is there a way to re-run the tests without pushing another commit? The build was broken when I submitted the change, but the build has been fixed and the test hasn't re-run. |
I'm getting merge conflicts when I try to merge locally. Could we rebase this and see where we stand? |
Yes. Absolutely. I’ll rebase it today and check on the None.None error
… On Sep 17, 2019, at 1:02 PM, Mitch Garnaat ***@***.***> wrote:
I'm getting merge conflicts when I try to merge locally. Could we rebase this and see where we stand?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
@garnaat this example looks completely broken. I'm okay re-writing it for CDK 1.8 and submitting a fresh PR. |
Title does not follow the guidelines of Conventional Commits. Please adjust title before merge. |
Issue #, if available:
*Description of changes: Created a version of the MyWidgetService that uses API Gateway Service Integrations to directly interact with S3.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.